Skip to content

Load boxes.yaml and allow vagrant-hostmanager to update hypervisor#500

Open
ianballou wants to merge 1 commit intotheforeman:masterfrom
ianballou:Vagrantfile-hostmanager-configure-host
Open

Load boxes.yaml and allow vagrant-hostmanager to update hypervisor#500
ianballou wants to merge 1 commit intotheforeman:masterfrom
ianballou:Vagrantfile-hostmanager-configure-host

Conversation

@ianballou
Copy link
Copy Markdown

Why are you introducing these changes? (Problem description, related links)

What are the changes introduced in this pull request?

  • Adds hostmanager config, mainly so that the hypervisor's /etc/hosts is updated along with the VMs' hosts files.
  • Increases the dev box disk size to 50 to allow more space for long-lasting logs and Pulp.
  • Allows users to create a boxes.yaml file to add to the default box list in the Vagrantfile.

How to test this pull request

Steps to reproduce:

  • Try to create a boxes.yaml like
---
# User-local box definitions (this file is gitignored)
# Each entry becomes a vagrant box. Available settings:
#   box:       base box (default: centos/stream9)
#   hostname:  FQDN (default: <name>.example.com)
#   memory:    RAM in MB (default: 2048)
#   cpus:      CPU count (default: 1)
#   disk_size: disk in GB (default: unset)

katello-production:
  memory: 12288
  cpus: 4
  disk_size: 50
  • Try vagrant-hostmanager and ensure the hypervisor's hosts file is updated.

Checklist

  • Tests added/updated (if applicable)
  • Documentation updated (if applicable)

@ianballou
Copy link
Copy Markdown
Author

How do we want to advertise this boxes.yaml file? Docs?

Also this fixes #437

@ehelms
Copy link
Copy Markdown
Member

ehelms commented May 7, 2026

The failure was due to #497

Comment thread Vagrantfile
end
end

# Load user-local box definitions from boxes.yaml (gitignored)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Reminds me we should clean up the gitignore as it has some forklift-isms.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants