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

Add bind-utils #173

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add bind-utils #173

wants to merge 1 commit into from

Conversation

johanburati
Copy link

Would be nice to have nslookup by default on the image.

Would be nice to have **nslookup** by default on the image.
@centos-ci
Copy link
Collaborator

Can one of the admins verify this patch?

@lpancescu
Copy link
Contributor

The Vagrant images are a minimal CentOS installation, only adding the few packages that are needed by Vagrant, like rsync. I'd rather keep the images minimal, please use Vagrant provisioning in your Vagrantfile to install any packages you need (nsloopup was declared obsolete by ISC in the favor of dig).

Anyway, the functionality you want is already available in the Vagrant images we have:

$ getent hosts sunsite.unc.edu
152.19.134.40   sunsite.unc.edu

@johanburati
Copy link
Author

johanburati commented Apr 28, 2020

@szarkos

Isn't this Kickstart used to create the OpenLogic:CentOS:8_1:latest image on Azure ?

# Kickstart for creating a CentOS 8 Azure VM

If that's the case it should be a standing replacement for the RedHat:RHEL:8.1:latest image.
Since the Redhat image has both nslookup and dig installed this image should as well imho.

Copy link
Contributor

@szarkos szarkos left a comment

Choose a reason for hiding this comment

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

LGTM

@szarkos
Copy link
Contributor

szarkos commented Apr 28, 2020

@johanburati - No, the OpenLogic team has their own repo (i.e. https://github.com/openlogic/AzureBuildCentOS/blob/master/ks/azure/centos81.ks), however, we should strive to keep all these in sync.

@N3WWN can you also add this change to OpenLogic's CenTOS 8.1 image?

@N3WWN
Copy link

N3WWN commented Apr 29, 2020

@johanburati / @szarkos

Seems like a reasonable request. 😉

I have add bind-utils in my local copy of the kickstart and will push the change when I refresh the CentOS 8.1 image shortly.

N3WWN added a commit to openlogic/AzureBuildCentOS that referenced this pull request Apr 29, 2020
…STYLE settings, config for point release builds and bind-utils per CentOS/sig-cloud-instance-build#173
N3WWN added a commit to openlogic/AzureBuildCentOS that referenced this pull request Apr 29, 2020
…B_TIMEOUT/GRUB_TIMEOUT_STYLE settings, config for point release builds and bind-utils (only on 8.1) per CentOS/sig-cloud-instance-build#173
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.

5 participants