Hey! I’ve submitted a PR to add terraform support for hyperdrive: https://github.com/cloudflare/terr

Hey! I’ve submitted a PR to add terraform support for hyperdrive: https://github.com/cloudflare/terraform-provider-cloudflare/pull/3111

However in our testing we’ve noticed that the API docs are inconsistent, they state that

password

password

password

password is a top level value (https://developers.cloudflare.com/api/operations/create-hyperdrive), in our testing we’ve found that it should be nested within

origin

origin

origin

origin.

Before I update my PR, and submit a new PR to the cloudflare-go repo, it would be good to clarify whether the API itself or the API docs are where the issue lies? I.e will the docs be updated to nest

password

password

password

password in

origin

origin

origin

origin? Or the API updated to expose

password

password

password

password at the top level?

GitHub

Add Hyperdrive Resource provider by carlos-alberto · Pull Request #...

Implements #3092
Requires cloudflare/cloudflare-go#1491 to be released.
Note: Not sure if the acctest is actually viable, as I believe DB creds need to actually be valid for the api request to work.

Cloudflare API Documentation

Interact with Cloudflare's products and services via the Cloudflare API

Ccarlos-alberto Hey! I’ve submitted a PR to add terraform support for hyperdrive: https://github...

Matt Silverlock•2/8/24, 4:08 PM

API docs are incorrect -

password

password

should be nested within the

origin

origin

object - e.g. https://github.com/cloudflare/workers-sdk/blob/d637bd59a8ea6612d59ed4b73e115287615e617d/packages/wrangler/src/hyperdrive/client.ts#L23

GitHub

workers-sdk/packages/wrangler/src/hyperdrive/client.ts at d637bd59a...

Home to Wrangler, the CLI for Cloudflare Workers® - cloudflare/workers-sdk

MMatt Silverlock API docs are incorrect - `password` should be nested within the `origin` object ...

carlos-albertoOP•2/8/24, 4:09 PM

Perfect thank you, will get a new PR up in cloudflare-go and update the terraform one

Ccarlos-alberto Perfect thank you, will get a new PR up in cloudflare-go and update the terrafor...

Matt Silverlock•2/8/24, 4:11 PM

The types there are fairly stable (beta notwithstanding; always risk of some changes)

MMatt Silverlock The types there are fairly stable (beta notwithstanding; always risk of some cha...

carlos-albertoOP•2/8/24, 4:30 PM

Gotcha,

Updated https://github.com/cloudflare/terraform-provider-cloudflare/pull/3111

And opened https://github.com/cloudflare/cloudflare-go/pull/1501

Thanks for your help

mrwind•2/8/24, 11:34 PM

is hyperdrive working with mongodb (on mongo atlas)?

Mmrwind is hyperdrive working with mongodb (on mongo atlas)?

Sam•2/9/24, 7:42 AM

No, it's not

Mmrwind is hyperdrive working with mongodb (on mongo atlas)?

Emilio•2/9/24, 9:03 PM

We currently support Postgres, and plan to support MySQL soon.

quambo•2/10/24, 11:51 AM

Anyone managed to hyperdrive working with remote database + kysely + local development

Using

pg@8.11.0

pg@8.11.0

, initiating kysely like this:

const connectionString =
      env.LOCAL_DATABASE_URL || env.HYPERDRIVE.connectionString;
new Kysely<DB>({
    dialect: new PostgresDialect({
      pool: new Pool({
        max: 1,
        connectionString,
        ssl: {
          rejectUnauthorized: false,
        },
      }),
    }),
  });

const connectionString =
      env.LOCAL_DATABASE_URL || env.HYPERDRIVE.connectionString;
new Kysely<DB>({
    dialect: new PostgresDialect({
      pool: new Pool({
        max: 1,
        connectionString,
        ssl: {
          rejectUnauthorized: false,
        },
      }),
    }),
  });

Using

LOCAL_DATABASE_URL

LOCAL_DATABASE_URL

for local development (connectionstring including

?sslmode=require

?sslmode=require

), I'm able to setup the connection, but any query will yield

Error: Connection terminated unexpectedly

Error: Connection terminated unexpectedly

.

Been trying to use

localConnectionString

localConnectionString

instead of bypassing hyperdrive in local dev, but it transforms my connectionstring to some

hyperdrive.local

hyperdrive.local

url.

Qquambo Anyone managed to hyperdrive working with remote database + kysely + local devel...

xatxat•2/10/24, 2:08 PM

i'm using the same approach with kysely + local development. works fine for me.

xatxat•2/10/24, 2:10 PM

i don't have

?sslmode=require

?sslmode=require

in my connection string

quambo•2/10/24, 2:34 PM

I just can't get it to work

trying postgres package, also no success, error

[wrangler:err] Error: internal error
    at cachedError (/x/node_modules/postgres/cf/src/query.js:170:23)
    at new Query (/x/node_modules/postgres/cf/src/query.js:36:24)
    at sql2 (/x/node_modules/postgres/cf/src/index.js:113:11)
    at Object.fetch (/x/index.ts:9:23)

[wrangler:err] Error: internal error
    at cachedError (/x/node_modules/postgres/cf/src/query.js:170:23)
    at new Query (/x/node_modules/postgres/cf/src/query.js:36:24)
    at sql2 (/x/node_modules/postgres/cf/src/index.js:113:11)
    at Object.fetch (/x/index.ts:9:23)

index.ts

import postgres from 'postgres';

export default {
  async fetch(request: Request, env: any) {
    const connectionString =
      env.DATABASE_URL || env.HYPERDRIVE.connectionString;
    console.log('fetch', request.url, env.DATABASE_URL);
    const sql = postgres(connectionString);
    const abc = await sql`SELECT * FROM tenant LIMIT 10`;
    console.log('abc', abc);

    return new Response(JSON.stringify({ ok: true }));
  },
};

import postgres from 'postgres';

export default {
  async fetch(request: Request, env: any) {
    const connectionString =
      env.DATABASE_URL || env.HYPERDRIVE.connectionString;
    console.log('fetch', request.url, env.DATABASE_URL);
    const sql = postgres(connectionString);
    const abc = await sql`SELECT * FROM tenant LIMIT 10`;
    console.log('abc', abc);

    return new Response(JSON.stringify({ ok: true }));
  },
};

package.json

{
  "name": "x",
  "dependencies": {
    "postgres": "^3.4.3"
  }
}

{
  "name": "x",
  "dependencies": {
    "postgres": "^3.4.3"
  }
}

wrangler.toml

name = "xyz"
compatibility_flags = ["nodejs_compat"]

name = "xyz"
compatibility_flags = ["nodejs_compat"]

quambo•2/10/24, 2:35 PM

i've been trying for hours to get this to work, but i keep on getting odd errors, with different wrangler versions and ssl true/false

quambo•2/10/24, 2:56 PM

wonder if its got to do with https://github.com/cloudflare/workerd/issues/1401

GitHub

🐛 BUG: Uncaught exception · Issue #1401 · cloudflare/workerd

Which Cloudflare product(s) does this pertain to? Wrangler or miniflare What version(s) of the tool(s) are you using? 3.5.0 What version of Node are you using? 18.16.1 What operating system are you...

Qquambo wonder if its got to do with https://github.com/cloudflare/workerd/issues/1401

Matt Silverlock•2/10/24, 5:54 PM

That’s an old issue that seems entirely unrelated.

Does your database accept TLS/SSL connections?

Qquambo I just can't get it to work 😦 trying postgres package, also no success, error `...

Matt Silverlock•2/10/24, 5:56 PM

Are your hyperdrive bindings configured too? When you create a Hyperdrive via the dash or wrangler, Hyperdrive itself connects to make sure it’s reachable before “saving” the configuration.

quambo•2/10/24, 6:59 PM

I found the reason @Matt.. after enabling logs, i could see certificate issues with my crunchybridge pg and the cf worker. Using

NODE_EXTRA_CA_CERTS=./assets/certificate.pem wrangler dev

NODE_EXTRA_CA_CERTS=./assets/certificate.pem wrangler dev

solved the issue.

EEmilio We currently support Postgres, and plan to support MySQL soon.

berkinovish•2/12/24, 3:50 PM

any ETA on when we can expect MySQL support?

Bberkinovish any ETA on when we can expect MySQL support?

Matt Silverlock•2/14/24, 11:13 AM

At this stage, closer to April/May.

moody 💭•2/14/24, 8:54 PM

hi, i'm running into issues with my rds database slowing down based on client write waits

moody 💭•2/14/24, 8:55 PM

is hyperdrive bandwidth throttled at all?

moody 💭•2/14/24, 8:55 PM

moody 💭•2/14/24, 9:00 PM

when this happens i see spikes in latency

Isaac McFadyen•2/14/24, 9:02 PM

According to the RDS docs this can also happen if there's high latency between the client and the database.

Isaac McFadyen•2/14/24, 9:02 PM

Are you accessing it via a Worker that's far from the DB?

moody 💭•2/14/24, 9:15 PM

i don't think so, my db is only in us-east-1 and i am noticing latency querying it from east coast

moody 💭•2/14/24, 9:16 PM

it seems like a download bandwidth issue on hyperdrive's side

moody 💭•2/14/24, 9:16 PM

i don't think i'm throttling the 10gbps network connection of the rds instance

moody 💭•2/14/24, 9:16 PM

and it's def not latency because it's only my really data heavy query

moody 💭•2/14/24, 9:17 PM

which btw also relates to a very high data egress bill from aws. i'm considering moving the compute to ec2 but it's an effort i'd rather not make right now

moody 💭•2/14/24, 9:18 PM

since i'll have to create an https cert and load balancer and some auto scaling stuff all the infra to deploy that in a pipeline, etc.

Unsmart•2/14/24, 9:21 PM

Not really anything cloudflare can do about AWS having high bandwidth pricing take that one up with AWS. But cloudflare doesnt throttle bandwidth usage... how much data are getting back and how long does it take to download? And whats the comparison to downloading direct from RDS?

moody 💭•2/14/24, 9:21 PM

found the bandwidth chart

moody 💭•2/14/24, 9:23 PM

10MB/s peak, i'm not sure where these clientwrite spikes could be coming from but i do cache in-memory on the worker so maybe workers are getting recycled

moody 💭•2/14/24, 9:24 PM

these queries aren't crazy, it's like 3 numeric fields with maybe 10k rows, probably O(MBs), i would use wire compression if that was a thing because one of the columns has a lot of repeated values