Skip to content

Conversation

xinfengliu
Copy link
Contributor

Fixes release CI job errors

Proposed Changes

Cherry-pick #457 and #458

Signed-off-by: Xinfeng Liu <XinfengLiu@icloud.com>
@xinfengliu xinfengliu marked this pull request as ready for review April 7, 2025 09:46
@xinfengliu xinfengliu requested a review from Copilot April 7, 2025 09:46
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (4)

.github/workflows/e2e.yml:8

  • Changing the OS version to ubuntu-22.04 may lead to differences in environment behavior. Verify that all workflow steps are compatible with this new image.
runs-on: ubuntu-22.04

.github/workflows/docs.yml:63

  • Upgrading to actions/upload-pages-artifact@v3 may introduce breaking changes. Please check the action's changelog to ensure its behavior meets the workflow requirements.
uses: actions/upload-pages-artifact@v3

.github/workflows/docs.yml:76

  • The update to actions/deploy-pages@v4 could affect the deployment process. It is recommended to review the updated version's documentation for any changes that may impact functionality.
uses: actions/deploy-pages@v4

.github/workflows/Release.yml:6

  • The addition of the workflow_dispatch trigger and explicit permissions should be reviewed to ensure they align with security best practices and the intended release process.
workflow_dispatch:

@xinfengliu xinfengliu requested a review from nwneisen April 8, 2025 07:10
@xinfengliu xinfengliu merged commit f9ec922 into Mirantis:master Apr 8, 2025
11 checks 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