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

initial commit for Cobbler provider #4271

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
4 participants
@thijsschnitger

This is an initial commit to get the provider we wrote for Cobbler merged as a builtin into Terraform core, as per ContainerSolutions/terraform-provider-cobbler#1.
I only included the .go files and left out documentation and example config because I thought you might want them elsewhere. Please let us know how to proceed.

@jtopjian

This comment has been minimized.

Show comment
Hide comment
@jtopjian

jtopjian Dec 12, 2015

Contributor

Very cool! I actually had this on my list of things to look at building. We use Cobbler heavily as our bare metal management utility.

I have a script to build out a simple, disposable Cobbler environment -- I was going to use it as the basis for provider acceptance tests. I'll dig it up, clean it up, and see if I can get this integrated.

(The acceptance tests wouldn't actually verify a provisioned bare metal server, but it can verify that a distro, profile, and system were correctly managed)

Contributor

jtopjian commented Dec 12, 2015

Very cool! I actually had this on my list of things to look at building. We use Cobbler heavily as our bare metal management utility.

I have a script to build out a simple, disposable Cobbler environment -- I was going to use it as the basis for provider acceptance tests. I'll dig it up, clean it up, and see if I can get this integrated.

(The acceptance tests wouldn't actually verify a provisioned bare metal server, but it can verify that a distro, profile, and system were correctly managed)

@thijsschnitger

This comment has been minimized.

Show comment
Hide comment
@thijsschnitger

thijsschnitger Dec 12, 2015

We built a Docker image for Cobbler which might be handy in this case. It's currently private but if you're interested we could make it public, it would need some sanatizing but it shouldn't be that much work.

We built a Docker image for Cobbler which might be handy in this case. It's currently private but if you're interested we could make it public, it would need some sanatizing but it shouldn't be that much work.

@jtopjian

This comment has been minimized.

Show comment
Hide comment
@jtopjian

jtopjian Jan 7, 2016

Contributor

Just wanted to touch base about this. I'm very sorry for the lack of communication -- I haven't had as much time as I'd like to work on Terraform.

I've been playing with this PR and cleaning up the code. I've started to do a refactor of the cobblerclient in order to handle GET/read requests. It's quite messy at the moment. Once I clean it up, I'll push it to github so you can see the changes. :)

Contributor

jtopjian commented Jan 7, 2016

Just wanted to touch base about this. I'm very sorry for the lack of communication -- I haven't had as much time as I'd like to work on Terraform.

I've been playing with this PR and cleaning up the code. I've started to do a refactor of the cobblerclient in order to handle GET/read requests. It's quite messy at the moment. Once I clean it up, I'll push it to github so you can see the changes. :)

@thijsschnitger

This comment has been minimized.

Show comment
Hide comment
@thijsschnitger

thijsschnitger Jan 7, 2016

Thanks for the heads up. If we can do anything in the mean time to speed up the process, please let us know!

Thanks for the heads up. If we can do anything in the mean time to speed up the process, please let us know!

@phinze

This comment has been minimized.

Show comment
Hide comment
@phinze

phinze Mar 17, 2016

Member

Hi @jtopjian - would love to get Cobbler support landed in Terraform core soon. 😀

Were you ever able to push the changes you referenced?

Member

phinze commented Mar 17, 2016

Hi @jtopjian - would love to get Cobbler support landed in Terraform core soon. 😀

Were you ever able to push the changes you referenced?

@jtopjian

This comment has been minimized.

Show comment
Hide comment
@jtopjian

jtopjian Mar 17, 2016

Contributor

I've got a patch in queue for the cobblerclient library. I need a couple hours to finish it up, $work has been keeping me away from it, but I'll try to get it done soonish.

Contributor

jtopjian commented Mar 17, 2016

I've got a patch in queue for the cobblerclient library. I need a couple hours to finish it up, $work has been keeping me away from it, but I'll try to get it done soonish.

@jtopjian jtopjian referenced this pull request Apr 1, 2016

Closed

Cobbler Provider #5969

@phinze

This comment has been minimized.

Show comment
Hide comment
@phinze

phinze Apr 16, 2016

Member

The Cobbler provider has landed via #5969 - thanks for getting things kicked off @thijsschnitger! 👍

Member

phinze commented Apr 16, 2016

The Cobbler provider has landed via #5969 - thanks for getting things kicked off @thijsschnitger! 👍

@phinze phinze closed this Apr 16, 2016

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