Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Environment Dependency Healthcheck #1653

Merged
merged 14 commits into from
Oct 23, 2019
Merged

Environment Dependency Healthcheck #1653

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
2945a8d
Add healthcheck for flow environment dependencies
joshmeek Oct 23, 2019
ce02278
Remove kubernetes dependency from docker storage
joshmeek Oct 23, 2019
1129130
Update docker storage test output
joshmeek Oct 23, 2019
c3984d6
Adjust environments and healthcheck for dynamic dependency specification
joshmeek Oct 23, 2019
6cb8072
Remove `egg` from Docker storage prefect install
joshmeek Oct 23, 2019
69b7282
Add tests for environment dependency health check
joshmeek Oct 23, 2019
41b4926
Update CHANGELOG
joshmeek Oct 23, 2019
601c003
Merge branch 'master' into healthchecks
joshmeek Oct 23, 2019
0a68f72
Update CHANGELOG
joshmeek Oct 23, 2019
6fdedf2
Update CHANGELOG
joshmeek Oct 23, 2019
b307a18
Add test for missing dependencies property and update healthcheck
joshmeek Oct 23, 2019
4b2aa38
Skip missing import test on python version < 3.6
joshmeek Oct 23, 2019
fc71da2
Update CHANGELOG
joshmeek Oct 23, 2019
a4c8707
Revert commented out debugging related agent functions
joshmeek Oct 23, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ These changes are available in the [master branch](https://github.com/PrefectHQ/

### Breaking Changes

- None
- `kubernetes` is no longer installed by default in deployed flow images - [#1653](https://github.com/PrefectHQ/prefect/pull/1653)

### Contributors

Expand Down
4 changes: 2 additions & 2 deletions src/prefect/agent/agent.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -153,8 +153,8 @@ def agent_process(self, tenant_id: str) -> bool:
)
)

self.update_states(flow_runs)
self.deploy_flows(flow_runs)
# self.update_states(flow_runs)
# self.deploy_flows(flow_runs)
joshmeek marked this conversation as resolved.
Show resolved Hide resolved
self.logger.info(
"Submitted {} flow run(s) for execution.".format(len(flow_runs))
)
Expand Down