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(3012): Trigger remote join job in restart when the last remote join job is successful #3132

Merged
merged 5 commits into from
Jun 6, 2024

Conversation

yakanechi
Copy link
Contributor

Context

Followup this PR
#3122

Remote Join job should be triggered on the new event on upstream when restart from an event downstream of the event in which Remote Join is successful.
However, the event and build are created but not triggered.

Objective

Fix the function call location so that the build can be fetched and triggered from the new event that was created.
In addition, add a test for this scenario. (In other words, make sure that it is triggered twice in total by start and restart)

References

License

I confirm that this contribution is made under a BSD license and that I have the authority necessary to make this contribution on behalf of its copyright owner.

@coveralls
Copy link

Coverage Status

coverage: 32.411% (-62.5%) from 94.938%
when pulling 914c0c9 on sonic-screwdriver-cd:remote-join-fix
into 23ee643 on screwdriver-cd:master.

@coveralls
Copy link

Coverage Status

coverage: 94.94% (+0.002%) from 94.938%
when pulling 18e9d80 on sonic-screwdriver-cd:remote-join-fix
into 23ee643 on screwdriver-cd:master.

yakanechi and others added 2 commits June 6, 2024 13:23
Co-authored-by: y-oksaku <43719835+y-oksaku@users.noreply.github.com>
Co-authored-by: y-oksaku <43719835+y-oksaku@users.noreply.github.com>
@coveralls
Copy link

Coverage Status

coverage: 94.94% (+0.002%) from 94.938%
when pulling 262ed58 on sonic-screwdriver-cd:remote-join-fix
into 23ee643 on screwdriver-cd:master.

@tkyi tkyi merged commit b29780f into screwdriver-cd:master Jun 6, 2024
2 checks passed
@yakanechi yakanechi deleted the remote-join-fix branch June 6, 2024 23:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants