Skip to content

Make agent set infrastructure_pid after starting a flow run #7613

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
Merged
merged 2 commits into from
Nov 22, 2022
Merged

Make agent set infrastructure_pid after starting a flow run #7613

merged 2 commits into from
Nov 22, 2022

Conversation

bunchesofdonald
Copy link
Contributor

Updates the agent to set the infrastructure_pid after starting a flow run. Attempt #2.

@netlify
Copy link

netlify bot commented Nov 21, 2022 •
edited
Loading

✅ Deploy Preview for prefect-orion ready!

Name Link
🔨 Latest commit 5703844
🔍 Latest deploy log https://app.netlify.com/sites/prefect-orion/deploys/637cf8b3d8057a0008faa671
😎 Deploy Preview https://deploy-preview-7613--prefect-orion.netlify.app/api-ref/prefect/agent
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@bunchesofdonald bunchesofdonald force-pushed the agent-infrastructure_pid branch from 26689b5 to 9356670 Compare November 22, 2022 15:05
zanieb
zanieb approved these changes Nov 22, 2022
View reviewed changes
Copy link
Contributor

@zanieb zanieb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice.

@bunchesofdonald bunchesofdonald force-pushed the agent-infrastructure_pid branch from ec45f01 to 5703844 Compare November 22, 2022 16:28
@bunchesofdonald bunchesofdonald changed the base branch from main to feature/flow-run-cancellation November 22, 2022 20:32
@bunchesofdonald bunchesofdonald merged commit 5a304bf into feature/flow-run-cancellation Nov 22, 2022
@bunchesofdonald bunchesofdonald deleted the agent-infrastructure_pid branch November 22, 2022 20:32
zanieb pushed a commit that referenced this pull request Nov 28, 2022
@bunchesofdonald @zanieb
Make agent set infrastructure_pid after starting a flow run (#7613)
42504c7
zanieb added a commit that referenced this pull request Nov 30, 2022
@zanieb
Make agent set infrastructure_pid after starting a flow run (#7613)
529d9a4
zanieb pushed a commit that referenced this pull request Nov 30, 2022
@bunchesofdonald @zanieb
Make agent set infrastructure_pid after starting a flow run (#7613)
6164f0d
zanieb pushed a commit that referenced this pull request Dec 1, 2022
@bunchesofdonald @zanieb
Make agent set infrastructure_pid after starting a flow run (#7613)
847040d
zanieb pushed a commit that referenced this pull request Dec 1, 2022
@bunchesofdonald @zanieb
Make agent set infrastructure_pid after starting a flow run (#7613)
6f598b0
github-actions bot pushed a commit that referenced this pull request Dec 1, 2022
@bunchesofdonald @github-actions
Make agent set infrastructure_pid after starting a flow run (#7613)
d810292
github-actions bot pushed a commit to ddelange/prefect that referenced this pull request Dec 15, 2022
@bunchesofdonald @github-actions
Make agent set infrastructure_pid after starting a flow run (Prefec…
7093b20 
…tHQ#7613)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
2 more reviewers

@zanieb zanieb zanieb approved these changes

@peytonrunyan peytonrunyan peytonrunyan approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@bunchesofdonald @zanieb @peytonrunyan