Thanks to visit codestin.com
Credit goes to github.com

Skip to content

tests: fix failing rabbitmq service #3376

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
May 13, 2025
Merged

tests: fix failing rabbitmq service #3376

merged 1 commit into from
May 13, 2025

Conversation

kayrus
Copy link
Contributor

@kayrus kayrus commented May 13, 2025

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 edit:testinfra This PR updates testing infrastructure code semver:patch No API change labels May 13, 2025
@coveralls
Copy link

coveralls commented May 13, 2025 •
edited
Loading

Coverage Status

coverage: 63.82%. remained the same
when pulling c4a191e on kayrus:devstack
into c9a3171 on gophercloud:main.

@kayrus kayrus force-pushed the devstack branch 6 times, most recently from c30870b to 02ac4b3 Compare May 13, 2025 13:03
@kayrus kayrus mentioned this pull request May 13, 2025
Merged
@kayrus kayrus force-pushed the devstack branch 2 times, most recently from bc9cc32 to 74336fb Compare May 13, 2025 13:55
@kayrus
Copy link
Contributor Author

kayrus commented May 13, 2025

@stephenfin

$ sudo cat /etc/hosts
127.0.0.1 localhost

# The following lines are desirable for IPv6 capable hosts
::1     localhost ip6-localhost ip6-loopback
fe00::0 ip6-localnet
ff00::0 ip6-mcastprefix
ff02::1 ip6-allnodes
ff02::2 ip6-allrouters
ff02::3 ip6-allhosts
-e 10.1.0.192 pkrvmwou917u6c3.vluyefalokqupermxzuhduojkb.gx.internal.cloudapp.net pkrvmwou917u6c3

wth? may it be the broken azure VM automation?

@kayrus kayrus force-pushed the devstack branch 2 times, most recently from 54a8dd1 to 01ad9d9 Compare May 13, 2025 15:24
@kayrus
Copy link
Contributor Author

kayrus commented May 13, 2025

should be fixed with the https://github.com/gophercloud/devstack-action/releases/tag/v0.17

stephenfin
stephenfin requested changes May 13, 2025
View reviewed changes
Copy link
Contributor

@stephenfin stephenfin left a comment

Choose a reason for hiding this comment

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

You missed one

@stephenfin stephenfin merged commit 5631f7c into gophercloud:main May 13, 2025
10 checks passed
@kayrus kayrus deleted the devstack branch May 13, 2025 16:19
@kayrus kayrus added the backport-v2 This PR will be backported to v2 label May 13, 2025
@gophercloud-backport-bot gophercloud-backport-bot bot mentioned this pull request May 13, 2025
Merged
stephenfin
stephenfin reviewed May 13, 2025
View reviewed changes
Copy link
Contributor

@stephenfin stephenfin left a comment

Choose a reason for hiding this comment

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

There's a bug here. Annoyingly GitHub doesn't report it as such (see https://github.com/orgs/community/discussions/114370)

.github/workflows/functional-baremetal.yaml
@@ -31,7 +31,7 @@ jobs:
- name: Work around broken dnsmasq
run: sudo apt-get purge -y dnsmasq-base
- name: Deploy devstack
uses: EmilienM/devstack-action@e82a9cbead099cba72f99537e82a360c3e319c69
uses: gophercloud@devstack-action@v0.17
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
uses: gophercloud@[email protected]
uses: gophercloud/[email protected]

@stephenfin
Copy link
Contributor

I am fixing as part of #3377.

@stephenfin stephenfin mentioned this pull request May 13, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stephenfin stephenfin stephenfin approved these changes

Assignees
No one assigned
Labels
backport-v2 This PR will be backported to v2 edit:actions This PR updates GitHub Actions code edit:testinfra This PR updates testing infrastructure code semver:patch No API change
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@kayrus @coveralls @stephenfin