up
Feed Advanced Search
Today
Today
Yesterday
Yesterday
natekupp updated the diff for D1286: [dagster-aws] Increase timeout on dagster-aws info CLI command.
up
natekupp retitled D1286: [dagster-aws] Increase timeout on dagster-aws info CLI command from [dagster-aws] Increase timeout on dagster-aws info CLI command to 500ms to [dagster-aws] Increase timeout on dagster-aws info CLI command.
Mon, Oct 14
Mon, Oct 14
Sat, Oct 12
Sat, Oct 12
LGTM, but see one comment
Fri, Oct 11
Fri, Oct 11
natekupp retitled D1269: Fix Spark configs duplication (#1823) from Fix Spark docs duplication (#1823) to Fix Spark configs duplication (#1823).
natekupp added a reviewer for D1267: [dagster-aws] Permit switching repository to a different dir (#1826): Restricted Project.
natekupp updated the diff for D1267: [dagster-aws] Permit switching repository to a different dir (#1826).
up
natekupp retitled D1267: [dagster-aws] Permit switching repository to a different dir (#1826) from [dagster-aws] Permit switching repository to a different dir to [dagster-aws] Permit switching repository to a different dir (#1826).
Thu, Oct 10
Thu, Oct 10
Mon, Oct 7
Mon, Oct 7
natekupp added a comment to D1237: Add dagster schedule restart command.
This lgtm, but can you add a test for this?
Sun, Oct 6
Sun, Oct 6
natekupp resigned from D1156: Remove -qqq from dev install script.
Can abandon this one now
Fri, Oct 4
Fri, Oct 4
natekupp added a comment to D1221: add client-side compute log size cap.
should we continue to stream logs but only retain/show the last X MB, similar to Buildkite?
natekupp updated the diff for D1224: [Makefile] Add install target which pip installs in verbose mode.
up
natekupp updated the diff for D1220: [dagster-aws] Add tests for ensure requirements.
rebase
natekupp updated the diff for D1215: [dagster-aws] Fix issue with keys sticking around.
py2
natekupp updated the diff for D1215: [dagster-aws] Fix issue with keys sticking around.
up
natekupp updated the diff for D1215: [dagster-aws] Fix issue with keys sticking around.
up
natekupp updated the diff for D1215: [dagster-aws] Fix issue with keys sticking around.
permissions
natekupp updated the diff for D1215: [dagster-aws] Fix issue with keys sticking around.
maybe maybe
natekupp updated the diff for D1215: [dagster-aws] Fix issue with keys sticking around.
ensure SSH is running on buildkite (maybe??)
natekupp updated the diff for D1215: [dagster-aws] Fix issue with keys sticking around.
clean up
natekupp updated the diff for D1215: [dagster-aws] Fix issue with keys sticking around.
well this was fun
natekupp updated the summary of D1217: [dagster-aws] Add URL to dagster-aws info.
natekupp updated the diff for D1217: [dagster-aws] Add URL to dagster-aws info.
success -> available
natekupp updated the summary of D1217: [dagster-aws] Add URL to dagster-aws info.
natekupp added inline comments to D1215: [dagster-aws] Fix issue with keys sticking around.
natekupp added inline comments to D1207: Add scheduler docs.
Thu, Oct 3
Thu, Oct 3
natekupp updated the diff for D1196: [dagster-aws] Only install dagster/dagit during dagster-aws up w/ requirements.txt parsing.
up
natekupp updated the diff for D1196: [dagster-aws] Only install dagster/dagit during dagster-aws up w/ requirements.txt parsing.
up
natekupp updated the diff for D1196: [dagster-aws] Only install dagster/dagit during dagster-aws up w/ requirements.txt parsing.
up
natekupp updated the diff for D1196: [dagster-aws] Only install dagster/dagit during dagster-aws up w/ requirements.txt parsing.
up
natekupp added a reviewer for D1182: [dagster-aws] CLI sync dagster.yaml to configure for RDS usage: Restricted Project.
natekupp updated the diff for D1182: [dagster-aws] CLI sync dagster.yaml to configure for RDS usage.
up
natekupp updated the diff for D1187: [dagster-aws] Fix VPC selection.
rebase master
natekupp updated the summary of D1187: [dagster-aws] Fix VPC selection.
thanks for doing this!!! helpful for me having not kept up with the changes over in this corner of things :)
natekupp updated the diff for D1177: [dagster-aws] Add info command.
rebase on master
natekupp updated the summary of D1177: [dagster-aws] Add info command.
overall looks fine to me, but seems rather risky for the night before a release given all the threading business here... we should hammer on this hard tomorrow
Wed, Oct 2
Wed, Oct 2
natekupp updated the diff for D1161: [dagster-aws] clean -> delete, cleanup main init, create sg instead of re-use.
fix bad rebase
natekupp updated the diff for D1161: [dagster-aws] clean -> delete, cleanup main init, create sg instead of re-use.
rebase
natekupp updated the diff for D1163: Move everything to Buildkite ECR Docker image.
up
natekupp added a comment to D1156: Remove -qqq from dev install script.
k let me experiment and see if I can figure it out
natekupp added a comment to D1156: Remove -qqq from dev install script.
hmmm I prefer -qqq for Buildkite builds because otherwise it's really hard to see issues in the logs (they cross the 1MB+ boundary and get truncated) - but prefer dropping -qqq for local installs. is there a way we can have that as an optional flag to these? My knowledge of make is limited :)
gonna abandon this for now, will put up something else when we're ready to implement task-level retry semantics
yah I'm gonna abandon this for now, we can resurrect it if needed when we revisit PipelineRun vs. RunConfig
natekupp edited reviewers for D1159: [dagster-aws cli] Wait for EC2 instance status OK, added: Restricted Project; removed: schrockn.
natekupp updated the summary of D1159: [dagster-aws cli] Wait for EC2 instance status OK.