Skip to content
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.

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

ConfigDrive functionality broken for Xen/XCP-Ng #10318

Open
NuxRo opened this issue Feb 3, 2025 · 2 comments · May be fixed by #10322
Open

ConfigDrive functionality broken for Xen/XCP-Ng #10318

NuxRo opened this issue Feb 3, 2025 · 2 comments · May be fixed by #10322
Assignees
Labels
Milestone

Comments

@NuxRo
Copy link
Contributor

NuxRo commented Feb 3, 2025

problem

A VM with cloud-init and all the shebang deployed on a ConfigDrive-enabled network on CXP-Ng is not configured at all.
The CD is not even detected.

versions

ACS 4.20 XCP-Ng 8.2

The steps to reproduce the bug

  1. Create isolated network with ConfigDrive
  2. Deploy VM
  3. Check VM - it is not configured
    ...

What to do about it?

Research and fix.

@NuxRo NuxRo added the type:bug label Feb 3, 2025
@DaanHoogland DaanHoogland added this to the 4.20.1 milestone Feb 4, 2025
@vishesh92 vishesh92 linked a pull request Feb 4, 2025 that will close this issue
14 tasks
@vishesh92 vishesh92 linked a pull request Feb 4, 2025 that will close this issue
14 tasks
@NuxRo
Copy link
Contributor Author

NuxRo commented Feb 6, 2025

Ok, so I feel pretty dumb for not knowing this, but XenServer won't allow you to have more than 1 CD-ROM active at any one time.
This means the guest never sees the 2nd one we add for ConfigDrive.

Workaround: use a regular disk drive, labelled the same etc, cloud-init can deal with it.

This has just become a bigger job..

Might be an idea to implement same for KVM, too, for consistency.

@DaanHoogland
Copy link
Contributor

@vishesh92 , I see your implementation on 4.19. Given @NuxRo 's explanation above, should we move this to a later version?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants