Feed Advanced Search
Yesterday
Yesterday
catherinewu committed R1:fc371b609e1e: fix redis condition in helm requirements.yaml (authored by catherinewu).
fix redis condition in helm requirements.yaml
Thu, Feb 25
Thu, Feb 25
catherinewu requested review of D6663: fix redis condition in helm requirements.yaml.
Wed, Feb 24
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.
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
Mon, Feb 22
Mon, Feb 22
Fri, Feb 19
Fri, Feb 19
catherinewu added inline comments to D6519: [buildkite][core] Adding checks for `db.text("CURRENT_TIMESTAMP")`.
catherinewu published D6602: PickledObjectGCSIOManager retry on 403 too for review.
Wed, Feb 17
Wed, Feb 17
catherinewu accepted D6501: RFC: Remove functionality from k8s run launchers to reload the code pointer and image name before launching.
catherinewu added inline comments to D6416: website top-level headlines update.
catherinewu requested changes to D6519: [buildkite][core] Adding checks for `db.text("CURRENT_TIMESTAMP")`.
a few comments; afterwards, good to go
Tue, Feb 16
Tue, Feb 16
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
Mon, Feb 15
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.
Sat, Feb 13
Sat, Feb 13
catherinewu requested review of D6476: Add parameter in Helm chart to set whether redis is created within k8s.
Thu, Feb 11
Thu, Feb 11
catherinewu committed R1:9c707e35f937: GH issue 3610 (add redis to requirements.yaml) (authored by catherinewu).
GH issue 3610 (add redis to requirements.yaml)
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)
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
catherinewu updated the diff for D6279: GH issue 3609 (Expose labels field in celery config).
up
catherinewu updated the diff for D6279: GH issue 3609 (Expose labels field in celery config).
up
catherinewu updated the diff for D6279: GH issue 3609 (Expose labels field in celery config).
up
catherinewu updated the diff for D6279: GH issue 3609 (Expose labels field in celery config).
up
catherinewu updated the diff for D6279: GH issue 3609 (Expose labels field in celery config).
up
catherinewu updated the diff for D6279: GH issue 3609 (Expose labels field in celery config).
up
catherinewu updated the diff for D6279: GH issue 3609 (Expose labels field in celery config).
up
catherinewu updated the diff for D6279: GH issue 3609 (Expose labels field in celery config).
up
catherinewu updated the diff for D6279: GH issue 3609 (Expose labels field in celery config).
up
catherinewu updated the diff for D6279: GH issue 3609 (Expose labels field in celery config).
up
catherinewu updated the summary of D6281: GH issue 3610 (add redis to requirements.yaml).
catherinewu committed R1:bc4933d8fcd7: GH issue 3051 (Flower configuration - expose env vars) (authored by catherinewu).
GH issue 3051 (Flower configuration - expose env vars)
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)
catherinewu updated the diff for D6279: GH issue 3609 (Expose labels field in celery config).
up
catherinewu added reviewers for D6281: GH issue 3610 (add redis to requirements.yaml): nate, rexledesma.
catherinewu added a comment to D6279: GH issue 3609 (Expose labels field in celery config).
added tests
catherinewu requested review of D6281: GH issue 3610 (add redis to requirements.yaml).
catherinewu updated the diff for D6279: GH issue 3609 (Expose labels field in celery config).
up
catherinewu updated the diff for D6279: GH issue 3609 (Expose labels field in celery config).
up
catherinewu updated the diff for D6279: GH issue 3609 (Expose labels field in celery config).
up
catherinewu updated the diff for D6279: GH issue 3609 (Expose labels field in celery config).
up
catherinewu updated the diff for D6279: GH issue 3609 (Expose labels field in celery config).
rebase
catherinewu requested review of D6279: GH issue 3609 (Expose labels field in celery config).
catherinewu added a comment to D6399: [buildkite][core] Add BK script to check DB schemas for cross-DBAPI compatability.
sweet ✨
Tue, Feb 9
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."
Sat, Feb 6
Sat, Feb 6
Fri, Feb 5
Fri, Feb 5
Wed, Feb 3
Wed, Feb 3
catherinewu retitled D6280: GH issue 3051 (Flower configuration - expose env vars) from GH issue 3051 to GH issue 3051 (Flower configuration - expose env vars).
catherinewu updated the summary of D6280: GH issue 3051 (Flower configuration - expose env vars).
catherinewu updated the test plan for D6280: GH issue 3051 (Flower configuration - expose env vars).
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
catherinewu requested review of D6280: GH issue 3051 (Flower configuration - expose env vars).
catherinewu added a reviewer for D6278: [GH 3611] `YOUR_ENV_VAR` is leaking from helm defaults into user clusters: nate.
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.
Fri, Jan 29
Fri, Jan 29
Jan 29 2021
Jan 29 2021
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 26 2021
Jan 26 2021
feel free to merge after making one last formatting change
catherinewu accepted D6141: [dagstermill] Allow for tags & description to be passed into solids (issue 3290).
catherinewu requested changes to D5651: [Github Community] Add a PR template.
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 25 2021
Jan 25 2021
Jan 24 2021
Jan 24 2021
Increase UI sensor polling rate
catherinewu published D6134: Increase UI sensor polling rate for review.
Jan 23 2021
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 21 2021
Jan 21 2021
catherinewu accepted D6087: [docs] Adjusted documentation to reflect that solids can also be tagged (issue 3291).
catherinewu added inline comments to D5651: [Github Community] Add a PR template.
Jan 20 2021
Jan 20 2021
catherinewu added inline comments to D6025: [Documentation] Refresh the Contributing guides.
catherinewu added inline comments to D6025: [Documentation] Refresh the Contributing guides.
catherinewu added inline comments to D6025: [Documentation] Refresh the Contributing guides.
fix docs rendering
Jan 19 2021
Jan 19 2021
update k8s docs
Jan 17 2021
Jan 17 2021
catherinewu requested review of D6037: update k8s docs.
Jan 15 2021
Jan 15 2021
catherinewu added inline comments to D5949: Add an example of how to trigger a pipeline by graphQL.
Jan 14 2021
Jan 14 2021
catherinewu committed R1:3d5076a71ca1: Add mailing list subscription button (via Revue) (authored by catherinewu).
Add mailing list subscription button (via Revue)
catherinewu updated the diff for D5986: Add mailing list subscription button (via Revue).
update license
catherinewu updated the diff for D5986: Add mailing list subscription button (via Revue).
rebase
