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

Use ansible-ref: devel for docs build workflows #2187

Merged

Conversation

alinabuzachis
Copy link
Contributor

@alinabuzachis alinabuzachis commented Oct 22, 2024

SUMMARY

Use ansible-ref: devel for docs build workflows

See also ansible-collections/amazon.aws#2353 (comment)

This needs to be merged in order to reflect the change.

Because of this change ansible/ansible@4f6276d documentation build process was failing with:

  /home/runner/work/_temp/docsbuild/head/rst/collections/amazon/aws/aws_ec2_inventory.rst:309: WARNING: 'envvar' reference target not found: ANSIBLE_INVENTORY_CACHE [ref.envvar]
  /home/runner/work/_temp/docsbuild/head/rst/collections/amazon/aws/aws_ec2_inventory.rst:370: WARNING: 'envvar' reference target not found: ANSIBLE_INVENTORY_CACHE_CONNECTION [ref.envvar]
  /home/runner/work/_temp/docsbuild/head/rst/collections/amazon/aws/aws_ec2_inventory.rst:435: WARNING: 'envvar' reference target not found: ANSIBLE_INVENTORY_CACHE_PLUGIN [ref.envvar]
  /home/runner/work/_temp/docsbuild/head/rst/collections/amazon/aws/aws_ec2_inventory.rst:500: WARNING: 'envvar' reference target not found: ANSIBLE_INVENTORY_CACHE_PLUGIN_PREFIX [ref.envvar]
  /home/runner/work/_temp/docsbuild/head/rst/collections/amazon/aws/aws_ec2_inventory.rst:565: WARNING: 'envvar' reference target not found: ANSIBLE_INVENTORY_CACHE_TIMEOUT [ref.envvar]
  /home/runner/work/_temp/docsbuild/head/rst/collections/amazon/aws/aws_rds_inventory.rst:259: WARNING: 'envvar' reference target not found: ANSIBLE_INVENTORY_CACHE [ref.envvar]
  /home/runner/work/_temp/docsbuild/head/rst/collections/amazon/aws/aws_rds_inventory.rst:320: WARNING: 'envvar' reference target not found: ANSIBLE_INVENTORY_CACHE_CONNECTION [ref.envvar]
  /home/runner/work/_temp/docsbuild/head/rst/collections/amazon/aws/aws_rds_inventory.rst:385: WARNING: 'envvar' reference target not found: ANSIBLE_INVENTORY_CACHE_PLUGIN [ref.envvar]
  /home/runner/work/_temp/docsbuild/head/rst/collections/amazon/aws/aws_rds_inventory.rst:450: WARNING: 'envvar' reference target not found: ANSIBLE_INVENTORY_CACHE_PLUGIN_PREFIX [ref.envvar]
  /home/runner/work/_temp/docsbuild/head/rst/collections/amazon/aws/aws_rds_inventory.rst:515: WARNING: 'envvar' reference target not found: ANSIBLE_INVENTORY_CACHE_TIMEOUT [ref.envvar]
ISSUE TYPE
  • Bugfix Pull Request
  • Docs Pull Request
  • Feature Pull Request
  • New Module Pull Request
COMPONENT NAME
ADDITIONAL INFORMATION

Copy link
Contributor

Build succeeded.
https://ansible.softwarefactory-project.io/zuul/buildset/1699d67fd1b043a392bbe57197a1f07b

ansible-galaxy-importer FAILURE in 6m 14s (non-voting)
✔️ build-ansible-collection SUCCESS in 10m 15s

Copy link
Contributor

@tremble tremble left a comment

Choose a reason for hiding this comment

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

@alinabuzachis alinabuzachis changed the title Use ansible 2.17 for documentation build Use ansible-ref: devel for docs build workflows Oct 22, 2024
Copy link
Contributor

@felixfontein felixfontein left a comment

Choose a reason for hiding this comment

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

Copy link
Contributor

Build succeeded.
https://ansible.softwarefactory-project.io/zuul/buildset/a5067c49827f45e9a18a484e01aa3998

ansible-galaxy-importer FAILURE in 4m 58s (non-voting)
✔️ build-ansible-collection SUCCESS in 11m 02s

@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Oct 24, 2024
Copy link
Contributor

Build succeeded (gate pipeline).
https://ansible.softwarefactory-project.io/zuul/buildset/ccc9f1229be0414cad30ebc87aaa3370

✔️ ansible-galaxy-importer SUCCESS in 3m 45s (non-voting)
✔️ build-ansible-collection SUCCESS in 10m 19s

Copy link
Contributor

Pull request merge failed: Resource not accessible by integration, You may need to manually rebase your PR and retry.

@tremble tremble merged commit e2faf12 into ansible-collections:main Oct 24, 2024
64 of 67 checks passed
@tremble
Copy link
Contributor

tremble commented Oct 24, 2024

Manual merge needed because it touches workflows.

Copy link

Docs Build 📝

Thank you for contribution!✨

This PR has been merged and your docs changes will be incorporated when they are next published.

alinabuzachis added a commit to abikouo/community.aws that referenced this pull request Oct 24, 2024
…s#2187)

* Use ansible-ref: devel for docs build workflows

Signed-off-by: Alina Buzachis <[email protected]>
alinabuzachis added a commit to mandar242/community.aws that referenced this pull request Oct 25, 2024
…s#2187)

* Use ansible-ref: devel for docs build workflows

Signed-off-by: Alina Buzachis <[email protected]>
alinabuzachis added a commit to GomathiselviS/community.aws that referenced this pull request Oct 25, 2024
…s#2187)

* Use ansible-ref: devel for docs build workflows

Signed-off-by: Alina Buzachis <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mergeit Merge the PR (SoftwareFactory)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants