Skip to content

tests: fix failing rabbitmq service#3376

Merged
stephenfin merged 1 commit intogophercloud:mainfrom
kayrus:devstack
May 13, 2025
Merged

tests: fix failing rabbitmq service#3376
stephenfin merged 1 commit intogophercloud:mainfrom
kayrus:devstack

Conversation

@kayrus
Copy link
Copy Markdown
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
Copy Markdown

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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
run: sudo apt-get purge -y dnsmasq-base
- name: Deploy devstack
uses: EmilienM/devstack-action@e82a9cbead099cba72f99537e82a360c3e319c69
uses: gophercloud@devstack-action@v0.17
Copy link
Copy Markdown
Contributor

@stephenfin stephenfin May 13, 2025

Choose a reason for hiding this comment

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

Suggested change
uses: gophercloud@devstack-action@v0.17
uses: gophercloud/devstack-action@v0.17

@stephenfin
Copy link
Copy Markdown
Contributor

stephenfin commented May 13, 2025

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