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

Staging v1.11.1 tasks #1812

Open
wants to merge 15 commits into
base: staging-v1.11.1
Choose a base branch
from
Open

Conversation

fernandofloresg
Copy link
Collaborator

@fernandofloresg fernandofloresg commented Nov 21, 2024

SUMMARY

Review release tasks for v1.11.1, this typically are reviewed in the merge against main, but since this is just a patch then this is just to have an extra set of eyes on this.

ISSUE TYPE
  • Release Pull Request
COMPONENT NAME
ADDITIONAL INFORMATION

@fernandofloresg fernandofloresg changed the base branch from staging-v1.11.0 to staging-v1.11.1 November 22, 2024 00:01
@fernandofloresg fernandofloresg marked this pull request as ready for review November 22, 2024 00:04
@fernandofloresg
Copy link
Collaborator Author

This shows ansible sanity failing but appears to be an issue with most recent PRs as you can see in the details is not related to the code. Executed it locally and does not fail, also, uploading this to the test servers works fine too.

(venv-2.17) ./ac --ac-sanity
-----------------------------------------------------------------------
Running ansible-test with managed python virtual environment: /Users/fernandoflores/Documents/IBM/Repos/ibm_zos_core/venv/venv-2.17.
-----------------------------------------------------------------------
Running sanity test "action-plugin-docs"
Running sanity test "ansible-doc"
Running sanity test "changelog"
Running sanity test "compile" on Python 3.12
Running sanity test "empty-init"
Running sanity test "ignores"
Running sanity test "import" on Python 3.12
Running sanity test "line-endings"
Running sanity test "no-assert"
Running sanity test "no-get-exception"
Running sanity test "no-illegal-filenames"
Running sanity test "no-smart-quotes"
Running sanity test "pep8"
Running sanity test "pslint"
Running sanity test "pylint"
Running sanity test "replace-urlopen"
Running sanity test "runtime-metadata"
Running sanity test "shebang"
Running sanity test "shellcheck"
Running sanity test "symlinks"
Running sanity test "use-argspec-type-path"
Running sanity test "use-compat-six"
Running sanity test "validate-modules"
Running sanity test "yamllint"

Copy link
Collaborator

@rexemin rexemin left a comment

Choose a reason for hiding this comment

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

Looks good, just noticed one wrong link

README.md Outdated Show resolved Hide resolved
@rexemin rexemin self-requested a review November 22, 2024 17:29
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