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

[PR #1270/4435c8a6 backport][stable-1.5] Disable galaxy-importer docs tests #1280

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Jun 29, 2022

This is a backport of PR #1270 as merged into main (4435c8a).

SUMMARY

Because galaxy-importer tests the collections in a sterile environment it can't read the shared amazon.aws fragments.

We're separately testing the docs more thoroughly using a github action, so we can disable the broken docs fragment testing via galaxy-importer

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

tests/galaxy-importer.cfg

ADDITIONAL INFORMATION

Disable galaxy-importer docs tests

SUMMARY
Because galaxy-importer tests the collections in a sterile environment it can't read the shared amazon.aws fragments.
We're separately testing the docs more thoroughly using a github action, so we can disable the broken docs fragment testing via galaxy-importer
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
tests/galaxy-importer.cfg
ADDITIONAL INFORMATION

(cherry picked from commit 4435c8a)
@ansibullbot ansibullbot added community_review feature This issue/PR relates to a feature request needs_triage new_contributor Help guide this first time contributor tests tests labels Jun 29, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

✔️ ansible-galaxy-importer SUCCESS in 5m 06s (non-voting)
✔️ build-ansible-collection SUCCESS in 5m 03s
✔️ ansible-test-sanity-docker-devel SUCCESS in 10m 39s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 9m 08s
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 10m 27s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 10m 58s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 8m 50s
ansible-test-sanity-docker-stable-2.13 FAILURE in 10m 50s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 5m 41s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 5m 47s

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 4m 53s (non-voting)
✔️ build-ansible-collection SUCCESS in 4m 59s
✔️ ansible-test-sanity-docker-devel SUCCESS in 10m 09s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 10m 10s
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 10m 16s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 11m 36s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 15s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 10m 00s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 5m 35s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 5m 00s

@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Jun 29, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 4m 03s (non-voting)
✔️ build-ansible-collection SUCCESS in 4m 53s
✔️ ansible-test-sanity-docker-devel SUCCESS in 8m 24s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 9m 57s
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 12m 13s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 9m 51s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 8m 56s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 9m 24s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 5m 27s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 5m 30s

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 9eb2d70 into stable-1.5 Jun 29, 2022
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-1.5/4435c8a64eada664d0d2cf742e8b2687058faf76/pr-1270 branch June 29, 2022 09:21
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
Sanity fix for changelog fragment

SUMMARY

Sanity fix for changelog fragment of ansible-collections#1275
The changelogs fragment is causing CI failure for sanity tests for other PRs such as ansible-collections#1253

ISSUE TYPE


Bugfix Pull Request

COMPONENT NAME

ADDITIONAL INFORMATION
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
…nsible-collections#1253)

route53_health_check: Fix "Name" tag key removal idempotentcy issue

SUMMARY
Depends-On: ansible-collections#1280

Fixes ansible-collections#1188

When using health_check_name as unique identifier (setting use_unique_names: True and providing a health_check_name) and health_check tags are set,
Current logic for adding name to a health_check causes an issue when rerunning the create/update task.
While ideally it should be idempotent, it removes the 'Name' tag (used for health_check_name) causing removal of health check name.
ISSUE TYPE


Bugfix Pull Request

COMPONENT NAME

route53_health_check
ADDITIONAL INFORMATION


To test, run the following sample playbook task twice
Expected output: Health check name should not disapper (i.e. 'Name' tag should not get removed on rerun)

---
- hosts: localhost
  gather_facts: False
  tasks:
    - name: Create a health-check
      amazon.aws.route53_health_check:
        health_check_name: my-test-hc
        use_unique_names: true
        fqdn: my-test-xyz.com
        type: HTTPS
        resource_path: /
        request_interval: 30
        failure_threshold: 3
        tags:
          Service: my-service
          Owner: my-test-xyz
          Lifecycle: dev

Reviewed-by: Mark Chappell <None>
Reviewed-by: GomathiselviS <None>
Reviewed-by: Mandar Kulkarni <mandar242@gmail.com>
Reviewed-by: Mike Graves <mgraves@redhat.com>
Reviewed-by: Gonéri Le Bouder <goneri@lebouder.net>
Reviewed-by: Alina Buzachis <None>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community_review feature This issue/PR relates to a feature request mergeit Merge the PR (SoftwareFactory) needs_triage new_contributor Help guide this first time contributor tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants