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

Bug: CI Allows merge of incomplete changes #3253

Closed
theunrepentantgeek opened this issue Sep 4, 2023 · 2 comments · Fixed by #3404
Closed

Bug: CI Allows merge of incomplete changes #3253

theunrepentantgeek opened this issue Sep 4, 2023 · 2 comments · Fixed by #3404
Assignees
Labels
high-priority Issues we intend to prioritize (security, outage, blocking bug)
Milestone

Comments

@theunrepentantgeek
Copy link
Member

Describe the bug

In #3206, we merged an update to cache without fully updating the index files used by our documentation. The docs were missing links to the samples.

Every CI build since then has failed because files are detected as modified during the CI process.

This is a bug in our CI process - this should not have been able to occur.

To Reproduce

Unclear how to reproduce, as I thought we'd fixed this. It may be due to @matthchr working from a fork.

Expected behavior

After merge of any PR, running task ci on main should not produce any modified files.

@theunrepentantgeek theunrepentantgeek added high-priority Issues we intend to prioritize (security, outage, blocking bug) and removed needs-triage 🔍 labels Sep 11, 2023
@theunrepentantgeek theunrepentantgeek added this to the v2.4.0 milestone Sep 11, 2023
@theunrepentantgeek
Copy link
Member Author

Flagged as high-priority because, while rare, when this happens it stalls all CI builds until fixed.

@theunrepentantgeek
Copy link
Member Author

I believe I've identified the scenario:

  • Merge a PR adding a new resource
  • Our documentation index pages will not have links to the CRD docs because they haven't been generated
  • Scheduled action creates a PR updating CRD docs
  • Merge CRD Docs
  • The branch main is now broken - when the generator is run again, the index pages will be modified to include links to the newly generated CRD documentation.

I'm testing this scenario to confirm.

@theunrepentantgeek theunrepentantgeek moved this from Backlog to In Progress in Azure Service Operator Roadmap Sep 28, 2023
@github-project-automation github-project-automation bot moved this from In Progress to Recently Completed in Azure Service Operator Roadmap Oct 12, 2023
@theunrepentantgeek theunrepentantgeek self-assigned this Oct 13, 2023
@matthchr matthchr moved this from Recently Completed to Ready for Release in Azure Service Operator Roadmap Oct 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
high-priority Issues we intend to prioritize (security, outage, blocking bug)
Projects
Development

Successfully merging a pull request may close this issue.

1 participant