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

CI doesn't validate changes pushed to master #47261

Closed
Youssef1313 opened this issue Aug 29, 2020 · 2 comments
Closed

CI doesn't validate changes pushed to master #47261

Youssef1313 opened this issue Aug 29, 2020 · 2 comments

Comments

@Youssef1313
Copy link
Member

Youssef1313 commented Aug 29, 2020

Ok, I think I don't know the terminologies very well. But here is the thing:

  1. Two PRs, "A" and "B", are created from master.
  2. PR "B" got merged to master.
  3. The branch of PR "A" builds successfully on its own. But it won't build successfully when merged with master because of changes introduced in PR "B". However, the CI passes for PR "A", and errors are only discovered after merging to master.

I think that's what happened with #47222 and #46949 (the branch of #47222 doesn't have the new file created in #46949)

The CI passed with #47222, but the master has build error after merging it. (see fd0e852)

If my analysis is correct, you may consider updating the CI to somehow build the result of merging the PR with master.

@sharwell
Copy link
Member

This is a timing issue. If the build for A starts before B gets merged, its results can be stale even though success is reported.

@Youssef1313
Copy link
Member Author

@sharwell Ah I see. Thanks a lot!

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

No branches or pull requests

2 participants