Skip to content

[v2] tests: fix failing rabbitmq service#3378

Merged
kayrus merged 1 commit intov2from
bp-v2-c4a191e
May 13, 2025
Merged

[v2] tests: fix failing rabbitmq service#3378
kayrus merged 1 commit intov2from
bp-v2-c4a191e

Conversation

@gophercloud-backport-bot
Copy link
Copy Markdown

Backport: #3376

Fixes #[PUT ISSUE NUMBER HERE]

Links to the line numbers/files in the OpenStack source code that support the
code in this PR:

[PUT URLS HERE]

@github-actions github-actions bot added edit:actions This PR updates GitHub Actions code v2 This PR targets v2 edit:testinfra This PR updates testing infrastructure code semver:patch No API change labels May 13, 2025
@kayrus kayrus merged commit 8d6dabc into v2 May 13, 2025
15 checks passed
@kayrus kayrus deleted the bp-v2-c4a191e branch May 13, 2025 16:27
@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 78.618%. remained the same
when pulling fc94678 on bp-v2-c4a191e
into dd3f103 on v2.

@stephenfin
Copy link
Copy Markdown
Contributor

As with the original PR, there's a bug here.

@kayrus
Copy link
Copy Markdown
Contributor

kayrus commented May 13, 2025

@stephenfin how does it work then?

@stephenfin
Copy link
Copy Markdown
Contributor

stephenfin commented May 13, 2025

GitHub silently ignores the error in the workflow unless you click into the "Checks" tab. For example, see: https://github.com/gophercloud/gophercloud/actions/runs/15001729097 (or click the "Checks" tab and look for the ⚠️ icons on the left).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

edit:actions This PR updates GitHub Actions code edit:testinfra This PR updates testing infrastructure code semver:patch No API change v2 This PR targets v2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants