Skip to content

Add config_drive to server struct#3536

Merged
mandre merged 1 commit intogophercloud:mainfrom
xshyve:server/add-config-drive-to-response
Nov 3, 2025
Merged

Add config_drive to server struct#3536
mandre merged 1 commit intogophercloud:mainfrom
xshyve:server/add-config-drive-to-response

Conversation

@xshyve
Copy link
Copy Markdown
Contributor

@xshyve xshyve commented Oct 16, 2025

Adds config_drive to the regular server struct used for responses. (currently only part of createOpts) which leads to situation that one can create a server with a config_drive. But one can not verify / see that information in any kind of response.

This fixes #3534.

Fixes #3524

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

https://docs.openstack.org/api-ref/compute/#servers-servers

@github-actions github-actions bot added edit:compute This PR updates compute code semver:minor Backwards-compatible change backport-v2 This PR will be backported to v2 labels Oct 16, 2025
@xshyve xshyve force-pushed the server/add-config-drive-to-response branch from e858d73 to 56fd64b Compare November 3, 2025 08:33
Copy link
Copy Markdown
Contributor

@mandre mandre left a comment

Choose a reason for hiding this comment

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

Thanks for making the requested change and making the field a boolean. Let's just wait for CI to run.

Adds config_drive to the regular server struct used for responses.
(currently only part of createOpts) which leads to situation that one
can create a server with a config_drive. But one can not verify / see
that information in any kind of response.

This fixes gophercloud#3534.
@xshyve xshyve force-pushed the server/add-config-drive-to-response branch from 56fd64b to 62a7ff0 Compare November 3, 2025 09:28
@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 63.812% (-0.01%) from 63.823%
when pulling 62a7ff0 on xshyve:server/add-config-drive-to-response
into 4879eb5 on gophercloud:main.

@mandre mandre enabled auto-merge November 3, 2025 10:36
@mandre mandre added this pull request to the merge queue Nov 3, 2025
Merged via the queue into gophercloud:main with commit 5b43709 Nov 3, 2025
80 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.

Compute V2: Add ConfigDrive to server struct

4 participants