Skip to content

Add migrations list support in compute service - #3951

Merged
stephenfin merged 1 commit into
gophercloud:mainfrom
tobias-urdin:osmigrations
Aug 19, 2026
Merged

stephenfin merged 1 commit into
gophercloud:mainfrom
tobias-urdin:osmigrations

Conversation

@tobias-urdin

Copy link
Copy Markdown
Contributor

This adds migration list support for the compute
service based on the previous PR #3244

Fixes: #3243

@github-actions github-actions Bot added the edit:compute This PR updates compute code label Aug 17, 2026
@github-actions github-actions Bot added semver:minor Backwards-compatible change backport-v2 This PR will be backported to v2 labels Aug 17, 2026

@stephenfin stephenfin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks like there are a few changes needed here. You can run make lint to catch much of these locally. I have a few other captalisation-related things I'd like to see tweaked also.

Comment thread openstack/compute/v2/migrations/results.go Outdated
Comment thread openstack/compute/v2/migrations/results.go Outdated
Comment thread openstack/compute/v2/migrations/results.go Outdated
Comment thread openstack/compute/v2/migrations/results.go Outdated
@tobias-urdin

Copy link
Copy Markdown
Contributor Author

Looks like there are a few changes needed here. You can run make lint to catch much of these locally. I have a few other captalisation-related things I'd like to see tweaked also.

Thanks! I've updated based on suggestions and checked testing. Let's see how it goes.

stephenfin
stephenfin previously approved these changes Aug 17, 2026
@stephenfin

Copy link
Copy Markdown
Contributor

Assuming CI is happy, I'm good with this.

@stephenfin
stephenfin enabled auto-merge August 17, 2026 17:02
@stephenfin
stephenfin disabled auto-merge August 17, 2026 22:48
@coveralls

coveralls commented Aug 17, 2026 •

Copy link
Copy Markdown

Coverage Status

coverage: 63.559% (-0.003%) from 63.562% — tobias-urdin:osmigrations into gophercloud:main

This adds migration list support for the compute
service based on the previous PR gophercloud#3244

Fixes: gophercloud#3243
Co-Authored-By: avsmirnov <[email protected]>
Signed-off-by: Tobias Urdin <[email protected]>
@tobias-urdin

Copy link
Copy Markdown
Contributor Author

Another round of lint fixes.

@stephenfin
stephenfin enabled auto-merge August 19, 2026 17:26
@stephenfin
stephenfin added this pull request to the merge queue Aug 19, 2026
Merged via the queue into gophercloud:main with commit ab058bc Aug 19, 2026
70 checks passed
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:compute This PR updates compute code semver:minor Backwards-compatible change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add migrations list support in compute service

3 participants