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

Fixes BENTO-52 #180

Closed
wants to merge 1 commit into from
Closed

Fixes BENTO-52 #180

wants to merge 1 commit into from

Conversation

jperry
Copy link

@jperry jperry commented Feb 28, 2014

I confirmed building a box using packer and this cleanup.sh change fixes the resolv.conf issue.

@juliandunn
Copy link
Contributor

Thanks, that was probably my bad 😦

I'm thinking it is actually cleaner to add this to the CentOS ks.cfg as a "-kudzu" line item.

@jperry
Copy link
Author

jperry commented Feb 28, 2014

Would you prefer I move it or we'll go with this for now? What about the other items being removed?

@juliandunn
Copy link
Contributor

@jperry Can you move it? (And also scratch-build the CentOS 6 box to make sure that adding a "-kudzu" line won't break it, given that Kludgezu doesn't exist on that platform)

The other packages are no-ops, which is why I originally deleted it as part of the Veewee->Packer port and hence broke this.

@jperry
Copy link
Author

jperry commented Feb 28, 2014

Sure, I'll try and get to it today or this weekend. Thanks!

@juliandunn
Copy link
Contributor

I just fixed this with PR #183 and will rebuild the boxes; then you can test.

@juliandunn juliandunn closed this Mar 3, 2014
@jperry
Copy link
Author

jperry commented Mar 4, 2014

Thanks Julian, sorry I didn't get to this over the weekend.

@tamano
Copy link

tamano commented Mar 6, 2014

Thank you Julian and Jay, I had the same problem.

Btw Julian, did you already rebuild the box? I'm still having this problem.

@juliandunn
Copy link
Contributor

@tamano No, I haven't rebuilt the box yet. Part of it is that I think I broke some stuff in slimming down the ks.cfg. I will let you know.

@tamano
Copy link

tamano commented Mar 7, 2014

Thank you for your kindness, Julian.

@juliandunn
Copy link
Contributor

Boxes are all rebuilt and uploaded. Please test!

@tamano
Copy link

tamano commented Mar 10, 2014

@juliandunn I tried with "centos-5.10-i386_chef-provisionerless.box".
I had to add package install "gcc-c++" and "readline-devel" to my Chef recipe to make my program works, but everything else was perfect for me.
Thank you again.

@juliandunn
Copy link
Contributor

@tamano Yes, that was part of the ks.cfg minimization I did. The boxes now come only with the bare minimum needed to run the system. You have to install everything else yourself.

Glad to hear that the host-only-network fix works.

@tamano
Copy link

tamano commented Mar 10, 2014

@juliandunn I understood. And it's Ok if it's designed. Thank you.

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.

3 participants