-
Notifications
You must be signed in to change notification settings - Fork 1.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Universal Deploy #1819
Merged
Merged
Universal Deploy #1819
Changes from 1 commit
Commits
Show all changes
100 commits
Select commit
Hold shift + click to select a range
7d9652c
Still working on it
cicdw 4055987
Remove client method that i will no longer use
cicdw 94c2386
Merge branch 'local-labels' into deploy-2.0
cicdw 3ec48c0
Add working run_agent command for running cloud flows locally
cicdw 8adf268
Add a small docstring about the run_agent method
cicdw 1afc970
Merge branch 'master' into deploy-2.0
cicdw 8dcf31b
Merge branch 'master' into deploy-2.0
cicdw b82f14a
Begin reworking local agent
cicdw 34e3c0d
Remove unused local code
cicdw 7c8a670
Merge branch 'master' into deploy-2.0
cicdw fe690e8
Add missed docker files
cicdw 1603a0d
Rename LocalAgent DockerAgent
cicdw 55f8cd8
Rename Local Agent to Docker Agent in docs
joshmeek 46fd6ae
Implement Docker Agent tests and remove all references of runner_toke…
joshmeek e8f2e9d
Add an untested version of the local agent
cicdw e21bddb
Add working truly local agent
cicdw 53d4cd2
Always include 'local' label on local agents
cicdw f545d7c
Reset default logging level for agents
cicdw 87b1b3e
Move local agent logic
cicdw 665b5bd
Update references to old Local Agent in docs
joshmeek 54e829c
Add doc for Local Agent
joshmeek 497c4f8
Update Local Agent link in docstring
joshmeek 52b2d33
Add note to Local Agent doc about multiple labels for local Flows
joshmeek b7d9dac
Set PYTHONPATH to preserve imports
cicdw 55d9753
Slightly improved labeling
cicdw 658450d
Add import paths keyword to local agent
cicdw 9a54ddb
Add supervisor to requirements
cicdw b4affa7
Always conert agent labels to list
cicdw 02d4db4
Set flow run state to Failed if deploy error occurs in local agent
cicdw b7b2686
Version fix
cicdw 823167d
Add supervisor installation
cicdw cc0a403
Remove API option from CLI install for local agents
cicdw e67154e
Get local paths working on Mac
cicdw bbedda4
Fix formatting of cli options
cicdw c8a2b6e
Resolve merge conflicts with master
cicdw 2e99cb8
Finish resolving merge conflicts
cicdw 06d75d4
Remove unnecessary supervisor comment
cicdw 3ee2de7
Fix agent CLI commands
cicdw 94942da
Update help documentation
cicdw 9096234
Add some more local agent polish
cicdw 7c6ed25
Touch up type hints
cicdw 8533b25
Begin fixing tests
cicdw 7196084
Fix a few more tests
cicdw b4fcd9a
Stop providing auth token to subprocess environment
cicdw ac640b2
Change agent start default to local from docker
joshmeek 106dd13
Remove default for agent install CLI command
joshmeek 40fb697
Update CLI docstring
joshmeek 873a178
Merge branch 'master' into deploy-2.1
cicdw 3e8b550
Graceful retrieval of PWD
cicdw 6f0e2a7
Reimplement agent runner token verification
joshmeek c6fb25f
Reimplement runner token pytest fixture
joshmeek b25c950
Refactor deploy -> register in core flow
joshmeek 28e4dad
Update references of deploy to register
joshmeek 042171d
Update client docstring
joshmeek eb6a73f
Add tests for new local agent
joshmeek 1a9a113
Update agent cli tests for local agent
joshmeek fa6e661
Merge branch 'deploy-2.2' into register
joshmeek b337177
Update cloud doc structure
joshmeek 9218884
correct agent start docs
a2b3e23
Add deprecation warning to flow.deploy
joshmeek 2098ddf
Merge pull request #1789 from PrefectHQ/register
joshmeek a0d6ff1
Merge branch 'deploy-2.2' into deploy_docs
joshmeek b24238f
add local agent flow logging support
7ed22af
direct all logging to stdout (not stderr)
6cd9a38
Added documentation for user upstart with cloud
joshmeek 9ef1837
add local agent heartbeat test
114f750
honor existing python path + fix pwd assumption
e79e2d5
install extras for testing in appveyor
1bef402
Building blocks of deployment documentation
joshmeek 8acbf3d
Rearrange cloud docs and add templates for next steps
joshmeek 95ff47c
Merge branch 'deploy_docs' into deploy-2.2
joshmeek dacb5a2
Added list of onboard docs to first Cloud page
joshmeek 5446aee
Tweak documentation wording
zdhughes 65e87ea
Add Cloud Tutorial docs
joshmeek 131ae91
Resolve merge conflicts with master (I think)
cicdw 591eef7
Tidy up so that docs can be served
cicdw f4a7bca
Add auto-labeling of flow environments based on storage type
cicdw 9e26414
Add show_flow_logs to doc strings
cicdw 1d784cd
Make auto-labels more specific
cicdw ec42cf4
Auto label the local agent with the new storage labels
cicdw 8089f29
Fix tests caused by new autolabels on local agent
cicdw 4e25362
Merge pull request #1817 from PrefectHQ/storage-auto-labels
cicdw ab1f3f1
Merge branch 'master' into deploy-2.2
cicdw 866c5da
Skip graphviz test which fails on appveyor
cicdw 8aec281
Merge branch 'master' into deploy-2.2
cicdw bbcdf1d
Move skipifs down to the execution submodule of tests
cicdw 98a632c
Merge branch 'deploy-2.2' of https://github.com/PrefectHQ/prefect int…
cicdw 35f0cc0
Import pytest
cicdw c308899
Remove unnecessary import of google cloud storage
cicdw 4eaafee
Fix failing windows tests
cicdw e14bab7
Use pytestmark
cicdw 83c9548
Remove path validation at deserialiation time for local storage
cicdw dda3513
Ensure agent labels are always a list
cicdw a477f1d
Add new storage classes to API documentation
cicdw 89e2358
add show_flow_logs to run_agent fn
cbd71f0
Update CHANGELOG
joshmeek 4233aef
Resolve merge conflicts
joshmeek 9b96654
Update local agent doc to include new default labels
joshmeek 5cf1238
Update storage options doc with notes about auto labels
joshmeek d0f3d39
Update storage class docstrings with notes about auto labels
joshmeek File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
direct all logging to stdout (not stderr)
- Loading branch information
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we can solve this later after this PR, but it seems that this is hard coded to a debugging log level instead of deriving the level from the config