Skip to content

tests: fix containerinfra unit tests - #3899

Closed
kayrus wants to merge 1 commit into
gophercloud:mainfrom
kayrus:fix-containerinfra-tests
Closed

kayrus wants to merge 1 commit into
gophercloud:mainfrom
kayrus:fix-containerinfra-tests

Conversation

@kayrus

@kayrus kayrus commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Fixes #3898

@github-actions github-actions Bot added edit:actions This PR updates GitHub Actions code edit:testinfra This PR updates testing infrastructure code labels Jul 29, 2026
@kayrus
kayrus force-pushed the fix-containerinfra-tests branch from d3ae86b to a838922 Compare July 29, 2026 21:16
@github-actions github-actions Bot added semver:patch No API change backport-v2 This PR will be backported to v2 labels Jul 29, 2026
@coveralls

coveralls commented Jul 29, 2026 •

Copy link
Copy Markdown

Coverage Status

coverage: 63.517%. remained the same — kayrus:fix-containerinfra-tests into gophercloud:main

@kayrus
kayrus force-pushed the fix-containerinfra-tests branch 2 times, most recently from 8b5eca4 to 95f7e23 Compare July 29, 2026 21:23
@kayrus
kayrus force-pushed the fix-containerinfra-tests branch from 95f7e23 to 7cc4b67 Compare July 29, 2026 21:30
@winiciusallan

Copy link
Copy Markdown
Contributor

I don't know why it's going on with this test. Running locally with go 1.25.11, the tests pass. If I remove the last slash here, the tests start to fail with the method mismatch.

Running with go 1.26.5 either way fails.

@stephenfin

Copy link
Copy Markdown
Contributor

It looks like this was working on a reproducer rather than the fix. Given we've identified the actual fix (in #3946), I'm inclined to close this.

@stephenfin stephenfin closed this Aug 14, 2026
@kayrus
kayrus deleted the fix-containerinfra-tests branch August 20, 2026 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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

Development

Successfully merging this pull request may close these issues.

tests: containerinfra unit tests are broken

4 participants