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.

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

fix: Let agents fire storage (un)mount completion events even when coexisting with storage-proxy #1570

Merged
merged 2 commits into from
Sep 13, 2023

Conversation

fregataa
Copy link
Member

@fregataa fregataa commented Sep 12, 2023

When there is any cohabiting storage with agents, agents should skip mount/umount task and just produce succeed event rather than just return.

If #1560 merged earlier than this PR, this should be closed.

Checklist: (if applicable)

  • Milestone metadata specifying the target backport version

@fregataa fregataa added the urgency:4 As soon as feasible, implementation is essential. label Sep 12, 2023
@fregataa fregataa added this to the 23.09 milestone Sep 12, 2023
@fregataa fregataa self-assigned this Sep 12, 2023
@github-actions github-actions bot added the size:S 10~30 LoC label Sep 12, 2023
@achimnol achimnol changed the title fix: produce mount job success for agents with cohabiting storage fix: Fire storage (un)mount completion events from agents Sep 13, 2023
@achimnol achimnol changed the title fix: Fire storage (un)mount completion events from agents fix: Let agents fire storage (un)mount completion events even when coexisting with storage-proxy Sep 13, 2023
@achimnol
Copy link
Member

Let's apply this first and then look into #1560.

@achimnol achimnol added this pull request to the merge queue Sep 13, 2023
Merged via the queue into main with commit a832a33 Sep 13, 2023
21 checks passed
@achimnol achimnol deleted the fix/produce-event-when-its-cohabiting-storage branch September 13, 2023 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:S 10~30 LoC urgency:4 As soon as feasible, implementation is essential.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants