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: Fix automation for dist-docs #63

Merged
merged 1 commit into from
Feb 8, 2024
Merged

Conversation

almusil
Copy link
Contributor

@almusil almusil commented Feb 7, 2024

The automation for dist-docs was broken because of the PR action. Fix the automation, align it more with current state of OVN ci and change the branches that should be updated according to current supported branches.

The updated action takes less time to complete (~3 minutes per job) and the steps should be clearer.

The example job run can be seen at https://github.com/almusil/ovn-website/actions/runs/7812249154.
And the PRs that it created:
almusil#13
almusil#14
almusil#15
almusil#16
almusil#17
almusil#18
almusil#19

The automation for dist-docs was broken because of the PR action.
Fix the automation, align it more with current state of OVN ci
and change the branches that should be updated according to current
supported branches.

The updated action takes less time to complete (~3 minutes per job)
and the steps should be clearer.

Signed-off-by: Ales Musil <[email protected]>
@almusil almusil mentioned this pull request Feb 7, 2024
@almusil
Copy link
Contributor Author

almusil commented Feb 7, 2024

@putnopvut Please take a look

@putnopvut
Copy link
Contributor

Thanks Ales, I like the changes. Much cleaner now. I guess we'll need to update the matrix with each new OVN release? That's not too bad. I could potentially add that into the ovn-website-maintenance scripts so that it happens automatically when we create new releases.

@putnopvut putnopvut merged commit 5825c4c into ovn-org:main Feb 8, 2024
1 check passed
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.

2 participants