Page MenuHomeElementl

catherinewu (Catherine)
UserAdministrator

Projects

User does not belong to any projects.

User Details

User Since
Feb 24 2020, 6:17 PM (59 w, 6 d)
Roles
Administrator

Recent Activity

Thu, Apr 15

catherinewu accepted D7439: Update slack bot release message.

Macro pooh_eating_honey:

Thu, Apr 15, 12:15 AM
catherinewu accepted D7441: Remove references to #contributors.
Thu, Apr 15, 12:14 AM

Thu, Mar 25

catherinewu added inline comments to D7147: Allow global service account for Dagster Helm chart.
Thu, Mar 25, 9:17 PM
catherinewu added a comment to D7125: Roll user deployments if Helm values change.

looks good! can we run the integration tests too?

Thu, Mar 25, 9:13 PM

Mar 19 2021

catherinewu added inline comments to D7070: Update helm release cli to take chart path.
Mar 19 2021, 6:15 PM

Mar 18 2021

catherinewu accepted D6651: [RFC] Add Helm job to migrate Dagster instance.
Mar 18 2021, 6:57 AM
catherinewu accepted D6507: Split user deployments into Helm subchart.

amazing work!! two small notes, then good to go 💃

Mar 18 2021, 4:36 AM
catherinewu accepted D6881: Remove checksum annotations in Helm user code deployments.

nice find! doesnt seem like user deployment loads the instance configmap, so looks safe to me

Mar 18 2021, 3:54 AM

Mar 17 2021

catherinewu accepted D6989: Update Kubernetes deployment docs.

a few non-blocking comments. looks good!

Mar 17 2021, 11:13 PM
catherinewu added a comment to D6507: Split user deployments into Helm subchart.

i think we need an end-to-end integration test for "dagster-user-deployments": {"enabled": True, "enableSubchart": False} to be sure that launching runs w/o using subcharts continues to work

Mar 17 2021, 6:19 PM
catherinewu accepted D6888: Update user deployment subchart schema.
Mar 17 2021, 6:12 PM

Mar 16 2021

catherinewu requested changes to D6888: Update user deployment subchart schema.
Mar 16 2021, 4:35 AM
catherinewu added inline comments to D6507: Split user deployments into Helm subchart.
Mar 16 2021, 12:25 AM

Mar 9 2021

catherinewu added inline comments to D6651: [RFC] Add Helm job to migrate Dagster instance.
Mar 9 2021, 6:25 AM

Mar 3 2021

catherinewu accepted D6705: Move dagster-helm cli to Helm chart.
Mar 3 2021, 10:09 PM
catherinewu accepted D6761: Disable flaky daemon test.
Mar 3 2021, 4:52 PM
catherinewu resigned from D6707: Fully qualify all image names.

looks good to me, defer to @rexledesma's feedback about schema + template

Mar 3 2021, 7:21 AM
catherinewu resigned from D6706: Add support for pushing built images to ECR.

after rex's comments, looks good to me. going to resign so it's out of my queue

Mar 3 2021, 7:17 AM
catherinewu accepted D6691: Add Helm pytests in buildkite.
Mar 3 2021, 7:13 AM

Feb 27 2021

catherinewu closed D6663: fix redis condition in helm requirements.yaml.
Feb 27 2021, 6:27 AM
catherinewu committed R1:fc371b609e1e: fix redis condition in helm requirements.yaml (authored by catherinewu).
fix redis condition in helm requirements.yaml
Feb 27 2021, 6:27 AM

Feb 25 2021

catherinewu requested review of D6663: fix redis condition in helm requirements.yaml.
Feb 25 2021, 6:18 AM

Feb 24 2021

catherinewu added a comment to D6651: [RFC] Add Helm job to migrate Dagster instance.

Also wanted to second daniel's concern about running the migration while dagit is in use.

Feb 24 2021, 5:27 PM
catherinewu added a comment to D6651: [RFC] Add Helm job to migrate Dagster instance.

Another option is to ask users to connect to their cluster and manually run kubectl apply -f dagster-instance-migrate.yaml to create the migration job so that users aren't caught off guard? for example, some users may want to be extra safe and backup their database before migration in case something unexpected happens

Feb 24 2021, 5:24 PM

Feb 22 2021

catherinewu accepted D6630: Add granular node selectors for Celery queue deployment.

Macro pooh_eating_honey:

Feb 22 2021, 10:09 PM
catherinewu accepted D6609: Warn if you run dagit without DAGSTER_HOME set.
Feb 22 2021, 7:57 PM
catherinewu accepted D6519: [buildkite][core] Adding checks for `db.text("CURRENT_TIMESTAMP")`.
Feb 22 2021, 6:16 PM

Feb 19 2021

catherinewu added inline comments to D6519: [buildkite][core] Adding checks for `db.text("CURRENT_TIMESTAMP")`.
Feb 19 2021, 10:51 PM
catherinewu accepted D6416: website top-level headlines update.

Macro pooh_eating_honey:

Feb 19 2021, 8:09 PM
catherinewu accepted D6602: PickledObjectGCSIOManager retry on 403 too.
Feb 19 2021, 6:08 AM
catherinewu published D6602: PickledObjectGCSIOManager retry on 403 too for review.
Feb 19 2021, 6:08 AM
catherinewu accepted D6599: Fix dagit backfill page when no daemon settings are set.
Feb 19 2021, 4:42 AM

Feb 17 2021

catherinewu added inline comments to D6501: RFC: Remove functionality from k8s run launchers to reload the code pointer and image name before launching.
Feb 17 2021, 7:33 PM
catherinewu accepted D6501: RFC: Remove functionality from k8s run launchers to reload the code pointer and image name before launching.

Macro chefkiss:

Feb 17 2021, 7:32 PM
catherinewu added inline comments to D6416: website top-level headlines update.
Feb 17 2021, 7:12 PM
catherinewu accepted D6496: Upgrade to Helm 3.

Macro pooh_eating_honey:

Feb 17 2021, 7:07 PM
catherinewu requested changes to D6519: [buildkite][core] Adding checks for `db.text("CURRENT_TIMESTAMP")`.

a few comments; afterwards, good to go

Feb 17 2021, 5:31 PM

Feb 16 2021

catherinewu closed D6476: Add parameter in Helm chart to set whether redis is created within k8s.
Feb 16 2021, 8:53 PM
catherinewu committed R1:0d2124b99533: Add parameter in Helm chart to set whether redis is created within k8s (authored by catherinewu).
Add parameter in Helm chart to set whether redis is created within k8s
Feb 16 2021, 8:53 PM

Feb 15 2021

catherinewu retitled D6476: Add parameter in Helm chart to set whether redis is created within k8s from Add parameter in Helm chart to set whether redis is created w/in k8s to Add parameter in Helm chart to set whether redis is created within k8s.
Feb 15 2021, 5:59 PM

Feb 13 2021

catherinewu requested review of D6476: Add parameter in Helm chart to set whether redis is created within k8s.
Feb 13 2021, 12:09 AM

Feb 11 2021

catherinewu accepted D6426: Marks ``dagster new-repo`` as experimental..
Feb 11 2021, 6:50 PM
catherinewu closed D6281: GH issue 3610 (add redis to requirements.yaml).
Feb 11 2021, 5:47 PM
catherinewu committed R1:9c707e35f937: GH issue 3610 (add redis to requirements.yaml) (authored by catherinewu).
GH issue 3610 (add redis to requirements.yaml)
Feb 11 2021, 5:47 PM
catherinewu closed D6279: GH issue 3609 (Expose labels field in celery config).
Feb 11 2021, 8:51 AM
catherinewu committed R1:88907f9473fb: GH issue 3609 (Expose labels field in celery config) (authored by catherinewu).
GH issue 3609 (Expose labels field in celery config)
Feb 11 2021, 8:51 AM
catherinewu added a comment to D6279: GH issue 3609 (Expose labels field in celery config).

the changes after the "accept" review were only to add the tests in helm.py that check that the label is correctly applied to the deployment

Feb 11 2021, 8:50 AM
catherinewu updated the diff for D6279: GH issue 3609 (Expose labels field in celery config).

up

Feb 11 2021, 8:30 AM
catherinewu updated the diff for D6279: GH issue 3609 (Expose labels field in celery config).

up

Feb 11 2021, 8:29 AM
catherinewu updated the diff for D6279: GH issue 3609 (Expose labels field in celery config).

up

Feb 11 2021, 8:29 AM
catherinewu updated the diff for D6279: GH issue 3609 (Expose labels field in celery config).

up

Feb 11 2021, 8:28 AM
catherinewu updated the diff for D6279: GH issue 3609 (Expose labels field in celery config).

up

Feb 11 2021, 8:23 AM
catherinewu updated the diff for D6279: GH issue 3609 (Expose labels field in celery config).

up

Feb 11 2021, 8:21 AM
catherinewu updated the diff for D6279: GH issue 3609 (Expose labels field in celery config).

up

Feb 11 2021, 8:20 AM
catherinewu updated the diff for D6279: GH issue 3609 (Expose labels field in celery config).

up

Feb 11 2021, 8:07 AM
catherinewu updated the diff for D6279: GH issue 3609 (Expose labels field in celery config).

up

Feb 11 2021, 8:07 AM
catherinewu updated the diff for D6279: GH issue 3609 (Expose labels field in celery config).

up

Feb 11 2021, 8:05 AM
catherinewu updated the summary of D6281: GH issue 3610 (add redis to requirements.yaml).
Feb 11 2021, 7:44 AM
catherinewu closed D6280: GH issue 3051 (Flower configuration - expose env vars).
Feb 11 2021, 7:40 AM
catherinewu committed R1:bc4933d8fcd7: GH issue 3051 (Flower configuration - expose env vars) (authored by catherinewu).
GH issue 3051 (Flower configuration - expose env vars)
Feb 11 2021, 7:40 AM
catherinewu added a comment to D6280: GH issue 3051 (Flower configuration - expose env vars).

sent a snippet of user's code over Slack. yes, the flower env is different from the celery env in their case (also I think being maximally flexible for stuff like flower is preferable, so that we don't get in users' way)

Feb 11 2021, 7:40 AM
catherinewu updated the diff for D6279: GH issue 3609 (Expose labels field in celery config).

up

Feb 11 2021, 7:34 AM
catherinewu added reviewers for D6281: GH issue 3610 (add redis to requirements.yaml): nate, rexledesma.
Feb 11 2021, 7:32 AM
catherinewu added a comment to D6279: GH issue 3609 (Expose labels field in celery config).

added tests

Feb 11 2021, 7:30 AM
catherinewu requested review of D6281: GH issue 3610 (add redis to requirements.yaml).
Feb 11 2021, 7:29 AM
catherinewu updated the diff for D6279: GH issue 3609 (Expose labels field in celery config).

up

Feb 11 2021, 7:08 AM
catherinewu updated the diff for D6279: GH issue 3609 (Expose labels field in celery config).

up

Feb 11 2021, 7:07 AM
catherinewu updated the diff for D6279: GH issue 3609 (Expose labels field in celery config).

up

Feb 11 2021, 7:00 AM
catherinewu updated the diff for D6279: GH issue 3609 (Expose labels field in celery config).

up

Feb 11 2021, 6:34 AM
catherinewu updated the diff for D6279: GH issue 3609 (Expose labels field in celery config).

rebase

Feb 11 2021, 6:03 AM
catherinewu requested review of D6279: GH issue 3609 (Expose labels field in celery config).
Feb 11 2021, 5:10 AM
catherinewu added a comment to D6399: [buildkite][core] Add BK script to check DB schemas for cross-DBAPI compatability.

awesome ✨

Feb 11 2021, 3:19 AM

Feb 9 2021

catherinewu requested changes to D6301: [Community] Create a Dagster resources center.

Looks good so far! Sending it back to your queue for "Add video thumbnails and move the resources center to dagster.io."

Feb 9 2021, 3:01 AM

Feb 6 2021

catherinewu updated the title for P150 parallel pipeline from parallel solids to parallel pipeline.
Feb 6 2021, 12:36 AM

Feb 5 2021

catherinewu created P150 parallel pipeline.
Feb 5 2021, 11:54 PM

Feb 3 2021

catherinewu accepted D6258: [Docs] [examples] Adding solids to pipeline_tag page header & sidebar.
Feb 3 2021, 8:08 PM
catherinewu retitled D6280: GH issue 3051 (Flower configuration - expose env vars) from GH issue 3051 to GH issue 3051 (Flower configuration - expose env vars).
Feb 3 2021, 7:45 PM
catherinewu updated the summary of D6280: GH issue 3051 (Flower configuration - expose env vars).
Feb 3 2021, 7:43 PM
catherinewu updated the test plan for D6280: GH issue 3051 (Flower configuration - expose env vars).
Feb 3 2021, 7:43 PM
catherinewu closed D6278: [GH 3611] `YOUR_ENV_VAR` is leaking from helm defaults into user clusters.
Feb 3 2021, 7:42 PM
catherinewu committed R1:d784622c683b: [GH 3611] `YOUR_ENV_VAR` is leaking from helm defaults into user clusters (authored by catherinewu).
[GH 3611] `YOUR_ENV_VAR` is leaking from helm defaults into user clusters
Feb 3 2021, 7:42 PM
catherinewu requested review of D6280: GH issue 3051 (Flower configuration - expose env vars).
Feb 3 2021, 8:05 AM
catherinewu added a reviewer for D6278: [GH 3611] `YOUR_ENV_VAR` is leaking from helm defaults into user clusters: nate.
Feb 3 2021, 7:42 AM
catherinewu retitled D6278: [GH 3611] `YOUR_ENV_VAR` is leaking from helm defaults into user clusters from GH 3611 to [GH 3611] `YOUR_ENV_VAR` is leaking from helm defaults into user clusters.
Feb 3 2021, 7:42 AM
catherinewu requested review of D6278: [GH 3611] `YOUR_ENV_VAR` is leaking from helm defaults into user clusters.
Feb 3 2021, 7:13 AM

Jan 29 2021

catherinewu accepted D6214: fix bad args for schedule job state.

Macro pooh_eating_honey:

Jan 29 2021, 9:09 PM
catherinewu resigned from D6152: [dagit] Prompt to refresh config when repo is refreshed.

will defer to ben or sashank on the code review. the demo for this looks amazing and can't wait till users get to benefit from it 💃

Jan 29 2021, 2:08 AM

Jan 26 2021

catherinewu accepted D5651: [Github Community] Add a PR template.

feel free to merge after making one last formatting change

Jan 26 2021, 6:47 AM
catherinewu accepted D6141: [dagstermill] Allow for tags & description to be passed into solids (issue 3290).

Macro pooh_eating_honey:

Jan 26 2021, 6:35 AM
catherinewu requested changes to D5651: [Github Community] Add a PR template.
Jan 26 2021, 2:58 AM
catherinewu requested changes to D6141: [dagstermill] Allow for tags & description to be passed into solids (issue 3290).
Jan 26 2021, 2:50 AM
catherinewu requested changes to D6141: [dagstermill] Allow for tags & description to be passed into solids (issue 3290).

Awesome, looks great. one last change (and a missing .) and then it's good to ship

Jan 26 2021, 12:59 AM

Jan 25 2021

catherinewu requested changes to D6141: [dagstermill] Allow for tags & description to be passed into solids (issue 3290).
Jan 25 2021, 6:30 PM

Jan 24 2021

catherinewu closed D6134: Increase UI sensor polling rate.
Jan 24 2021, 1:02 AM
catherinewu committed R1:f2bd19e1b3d4: Increase UI sensor polling rate (authored by catherinewu).
Increase UI sensor polling rate
Jan 24 2021, 1:02 AM
catherinewu published D6134: Increase UI sensor polling rate for review.
Jan 24 2021, 12:45 AM

Jan 23 2021

catherinewu requested changes to D5651: [Github Community] Add a PR template.

requesting a few changes. next time, might be better to do this in a quip doc for faster iteration?

Jan 23 2021, 3:01 AM
catherinewu accepted D6025: [Documentation] Refresh the Contributing guides.
Jan 23 2021, 2:57 AM
catherinewu accepted D6122: Update url path deploying/k8s -> deploying/kubernetes.
Jan 23 2021, 2:55 AM
catherinewu accepted D6120: Update kubernetes deploy guide.

Macro pooh_eating_honey:

Jan 23 2021, 2:53 AM