Page MenuHomePhabricator
Feed Advanced Search

Yesterday

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

Thu, Feb 25

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

Wed, Feb 24

catherinewu added a comment to D6651: [RFC] Add Helm hook to migrate dagster instance.

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

Wed, Feb 24, 5:27 PM
catherinewu added a comment to D6651: [RFC] Add Helm hook 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

Wed, Feb 24, 5:24 PM

Mon, Feb 22

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

Macro pooh_eating_honey:

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

Fri, Feb 19

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

Macro pooh_eating_honey:

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

Wed, Feb 17

catherinewu added inline comments to D6501: RFC: Remove functionality from k8s run launchers to reload the code pointer and image name before launching.
Wed, Feb 17, 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:

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

Macro pooh_eating_honey:

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

a few comments; afterwards, good to go

Wed, Feb 17, 5:31 PM

Tue, Feb 16

catherinewu closed D6476: Add parameter in Helm chart to set whether redis is created within k8s.
Tue, Feb 16, 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
Tue, Feb 16, 8:53 PM

Mon, Feb 15

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.
Mon, Feb 15, 5:59 PM

Sat, Feb 13

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

Thu, Feb 11

catherinewu accepted D6426: Marks ``dagster new-repo`` as experimental..
Thu, Feb 11, 6:50 PM
catherinewu closed D6281: GH issue 3610 (add redis to requirements.yaml).
Thu, Feb 11, 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)
Thu, Feb 11, 5:47 PM
catherinewu closed D6279: GH issue 3609 (Expose labels field in celery config).
Thu, Feb 11, 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)
Thu, Feb 11, 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

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

up

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

up

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

up

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

up

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

up

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

up

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

up

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

up

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

up

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

up

Thu, Feb 11, 8:05 AM
catherinewu updated the summary of D6281: GH issue 3610 (add redis to requirements.yaml).
Thu, Feb 11, 7:44 AM
catherinewu closed D6280: GH issue 3051 (Flower configuration - expose env vars).
Thu, Feb 11, 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)
Thu, Feb 11, 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)

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

up

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

added tests

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

up

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

up

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

up

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

up

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

rebase

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

sweet ✨

Thu, Feb 11, 3:19 AM

Tue, Feb 9

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."

Tue, Feb 9, 3:01 AM

Sat, Feb 6

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

Fri, Feb 5

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

Wed, Feb 3

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

Fri, Jan 29

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

Macro pooh_eating_honey:

Fri, Jan 29, 9:09 PM

Jan 29 2021

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

Jan 21 2021

catherinewu accepted D6087: [docs] Adjusted documentation to reflect that solids can also be tagged (issue 3291).

Macro pooh_eating_honey:

Jan 21 2021, 7:41 PM
catherinewu added inline comments to D5651: [Github Community] Add a PR template.
Jan 21 2021, 2:38 AM

Jan 20 2021

catherinewu added inline comments to D6025: [Documentation] Refresh the Contributing guides.
Jan 20 2021, 11:16 PM
catherinewu added inline comments to D6025: [Documentation] Refresh the Contributing guides.
Jan 20 2021, 11:10 PM
catherinewu added inline comments to D6025: [Documentation] Refresh the Contributing guides.
Jan 20 2021, 4:35 AM
catherinewu closed D6051: fix docs rendering.
Jan 20 2021, 1:28 AM
catherinewu committed R1:e573617c2d19: fix docs rendering (authored by catherinewu).
fix docs rendering
Jan 20 2021, 1:28 AM

Jan 19 2021

catherinewu closed D6037: update k8s docs.
Jan 19 2021, 9:52 PM
catherinewu committed R1:e72bd1004000: update k8s docs (authored by catherinewu).
update k8s docs
Jan 19 2021, 9:51 PM
catherinewu accepted D6039: Default user deployment replica count to 1.

Macro pooh_eating_honey:

Jan 19 2021, 5:34 PM

Jan 17 2021

catherinewu requested review of D6037: update k8s docs.
Jan 17 2021, 12:55 AM
catherinewu abandoned D3939: add test for celery k8s run launcher termination without run launcher.
Jan 17 2021, 12:28 AM

Jan 15 2021

catherinewu added inline comments to D5949: Add an example of how to trigger a pipeline by graphQL.
Jan 15 2021, 2:34 AM

Jan 14 2021

catherinewu closed D5986: Add mailing list subscription button (via Revue).
Jan 14 2021, 9:49 PM
catherinewu committed R1:3d5076a71ca1: Add mailing list subscription button (via Revue) (authored by catherinewu).
Add mailing list subscription button (via Revue)
Jan 14 2021, 9:49 PM
catherinewu added a reviewer for D5986: Add mailing list subscription button (via Revue): yuhan.
Jan 14 2021, 9:26 PM
catherinewu updated the diff for D5986: Add mailing list subscription button (via Revue).

update license

Jan 14 2021, 9:26 PM
catherinewu updated the diff for D5986: Add mailing list subscription button (via Revue).

rebase

Jan 14 2021, 9:24 PM
catherinewu added a comment to D5986: Add mailing list subscription button (via Revue).

Jan 14 2021, 2:30 AM