User Details
- User Since
- Jan 19 2021, 9:41 PM (106 w, 5 d)
- Roles
- Disabled
Jul 28 2021
Jul 15 2021
Jul 13 2021
Updating to address @rexledesma's comments
Jul 12 2021
Jul 9 2021
lgtm besides the comment
Jul 8 2021
Jul 2 2021
lgtm
Jun 30 2021
Nice!
Jun 29 2021
Jun 21 2021
Lgtm! Nice find w/ pytest-rerun
Jun 18 2021
Based on discussion with @rexledesma, refactoring to just use Container as a schema
Allowing for imagePullPolicy as well
Sounds good to me! This would break run attribution in the demo, right, since that tries to read the header (as a proof of concept for the prezi ask, I think?)
Addressing 'TODO'
responding to @rexledesma's comments
Jun 17 2021
requiring config of helm_dir_path & subchart_paths via @rexledesma's comments
Also increase number of retires on helm checks since that appears to be a problem
Jun 10 2021
Jun 9 2021
more docstring edits
Fix docstrings
Jun 7 2021
This looks fine to me. I'll take a look at that Drizly problem ASAP.
lgtm! Haven't run into the Interface error personally but I figure if it's working locally for you it's fine!
Jun 3 2021
responding to @rexledesma's comments
Removing patch on warnings
Jun 2 2021
rebasing & prefixing test class with Test** to adhere to pytest conventions
prefixing with Test to adhere to pytest conventions
Jun 1 2021
rebasing
Renaming to QueuedRunCoordinatorTests as per @jordansanders' advice
Fix tox
Fix import
rename run_attribution -> run_attribution_example
restructuring to address @rexledesma's comments as well as rebasing
This issue was there to track a mypy problem because we had a pin in SqlAlchemy for <1.4 (I think related to this diff). I'll add in the type annotation in a separate diff - this LGTM. thanks for catching this Alex.
Oops, looks like I missed this - I'll work on getting these under test ASAP.
May 20 2021
Adding in asserts & cleaning up tests
Added tests & removed the NullPool wait_timeout fix - there's no pool for that case so there's nothing to recycle
May 19 2021
maybe fix lint before you push? otw lgtm.
May 17 2021
May 11 2021
Renaming to Dagster-Session-Token
rebasing & addressing comments
May 10 2021
May 7 2021
Oops, sorry for missing this! thanks for the fix.
Adding comment and renaming to headerAuthToken
adding comment
May 6 2021
Removing React.useMemo on rootServerURI as per @dish's suggestion
The removal of the GEventExecutor
I am pretty sure this is fine to do - and is effectively replicating the way the all-websocket approach worked - but we should add a test cases section specifically about making sure htttp requests can go through while the active run event log subscription is firing data back. You can do this by navigating around while another tab is actively watching a run of many_events from the toyes repo or something.
No longer overrwriting runs page on network errors
Apr 30 2021
Apr 29 2021
Removing automation change
Updating changelog
Apr 27 2021
Apr 26 2021
Reformatting reload_repository_location docs