oops, accidentally removed a flag
Feed All Stories
Today
Today
rexledesma published D6806: Fix helm-release cli for review.
fix indentation in versions.yaml
yuhan requested review of D6805: fix indentation in versions.yaml.
Allow config for custom instance classes
Factor instance weakref machinery into a mixin
sidkmenon added reviewers for D6784: [core] Adding destructor to SqlPollingEventWatcher: catherinewu, rexledesma, bob.
dish committed R1:245ee124968c: [dagit] Localize elapsed time and logs timestamps (authored by dish).
[dagit] Localize elapsed time and logs timestamps
fix unscheduled count for legacy backfills
[dagit] Move mode alongside pipeline name
dish added a comment to D6744: [dagit] Move mode alongside pipeline name.
I'm not sure how easy it'd be to retrieve repoAddress within the component...
[dagit] Allow Group wrapping
dish committed R1:def50f8da17c: [dagit] Separate pipeline and snapshot links in Run header (authored by dish).
[dagit] Separate pipeline and snapshot links in Run header
prha requested review of D6803: fix unscheduled count for legacy backfills.
remove tqdm unless printing
Chris DeCarolis <cdguitar817@gmail.com> committed R1:aacc40de5ba4: [4/n init_resources] init_resources context manager (authored by Chris DeCarolis <cdguitar817@gmail.com>).
[4/n init_resources] init_resources context manager
Yesterday
Yesterday
dgibson requested review of D6801: Add a DaemonRepositoryLocationOrigin.
alangenfeld added inline comments to D6797: Launch API.
CHANGES for 0.10.8
[docs] Instance and Daemon pages
cdecarolis added inline comments to D6797: Launch API.
alangenfeld requested review of D6782: Retries -> RetryMode & RetryState.
alangenfeld added inline comments to D6797: Launch API.
Harbormaster failed remote builds in B26881: Diff 32857 for D6787: [4/n init_resources] init_resources context manager!
cdecarolis added a comment to D6797: Launch API.
One thing that I'm concerned about is communicating the failure modes of the resolution mechanism that we're doing here. It's a bit of magic, and there's a lot of ways each can fail.
dgibson requested review of D6796: Re-enable flaky daemon test.
cdecarolis requested review of D6797: Launch API.
cdecarolis updated the diff for D6787: [4/n init_resources] init_resources context manager.
Up
cdecarolis updated the diff for D6787: [4/n init_resources] init_resources context manager.
Removed comment, removed recorder
sidkmenon requested review of D6784: [core] Adding destructor to SqlPollingEventWatcher.
Clean up observer in sqlite event log
prha requested review of D6794: remove tqdm unless printing.
dgibson requested review of D6790: Clean up observer in sqlite event log.
Harbormaster failed remote builds in B26866: Diff 32839 for D6768: Allow config for custom instance classes!
add chunking for synchronous backfill
max updated the diff for D6768: Allow config for custom instance classes.
add test and docs
schrockn added a comment to D6792: [dagit] Separate pipeline and snapshot links in Run header.
Nice. Would love this in the run list viewer too!
This seems like a nice presentation - I think we've been struggling for a while to find a consistent home for "mode" that doesn't make it look like a tag. I like having the <PipelineAndMode> component to wrap up the UI - I wonder if we could push the href generation down into that component in any way? I'm not sure how easy it'd be to retrieve repoAddress within the component...
Oh this is super cool - nice that we don't need moment for this sort of thing these days.
max requested review of D6793: Use ECR alias.
Harbormaster failed remote builds in B26858: Diff 32829 for D6764: Factor instance weakref machinery into a mixin!
max updated the diff for D6764: Factor instance weakref machinery into a mixin.
blk
dish requested review of D6792: [dagit] Separate pipeline and snapshot links in Run header.
dish requested review of D6791: [dagit] Allow Group wrapping.
prha requested review of D6777: added graphql-endpoints to wipe a single asset.
dgibson committed R1:38aa850efa19: [easy] Be less restrictive about origins coming into… (authored by dgibson).
[easy] Be less restrictive about origins coming into…
prha requested review of D6778: add wipe action to the new asset catalog (feature flagged).
dgibson committed R1:9631fd5333a1: Add 'use_ssl' boolean to grpc server workspace entry (authored by dgibson).
Add 'use_ssl' boolean to grpc server workspace entry
max updated the diff for D6764: Factor instance weakref machinery into a mixin.
add deprecation warning
prha requested review of D6789: add chunking for synchronous backfill.
alangenfeld resigned from D6768: Allow config for custom instance classes.
@alangenfeld any objections
max updated the diff for D6764: Factor instance weakref machinery into a mixin.
rebase and backcompat
johann added inline comments to D6765: RFC: cancel runs via daemon.
Harbormaster failed remote builds in B26849: Diff 32816 for D6771: [easy] Be less restrictive about origins coming into GrpcServerRepositoryLocaitonHandle!
Sidharth Menon <menon.sid.k@gmail.com> committed R1:768f2fe9c01c: [core][dagster-graphql] Fixing flaky dagster-graphql Polling Event Watcher tests (authored by Sidharth Menon <menon.sid.k@gmail.com>).
[core][dagster-graphql] Fixing flaky dagster-graphql Polling Event Watcher tests
dish committed R1:55803ecc23c1: [dagit] Bold object names in Schedules/Sensors tables (authored by dish).
[dagit] Bold object names in Schedules/Sensors tables
sidkmenon added a comment to D6783: [core][dagster-graphql] Fixing flaky dagster-graphql Polling Event Watcher tests.
flatten asset catalog table
dgibson added a comment to D6765: RFC: cancel runs via daemon.
Using a tag here seems OK to me. Adding tags can also be race-condition-y though... e.g. if this tag gets added while the run launcher is in the middle of adding other tags, bad things can happen (mostly since there are two sources of truth for the run tags - the serialized tags in the runs table, and the columns in the run tags table).