Skip to content
This repository has been archived by the owner on Jul 20, 2018. It is now read-only.

Convergence plans with joyent/docker-node team #77

Closed
rvagg opened this issue Jul 8, 2015 · 32 comments
Closed

Convergence plans with joyent/docker-node team #77

rvagg opened this issue Jul 8, 2015 · 32 comments

Comments

@rvagg
Copy link
Member

rvagg commented Jul 8, 2015

As per nodejs/docker-node#28 I've added @chorrell and @dcrudgington to the @nodejs/docker team. Handing over to y'all now to discuss how to converge your resources to continue to produce both images but as a combined team. Timing and the details of how you manage both repos are entirely up to you of course. As usual if you need any resources from the Build WG then just @-mention us and we'll come running.

Happy times!

@rvagg
Copy link
Member Author

rvagg commented Jul 8, 2015

Also going to ping @mhart who maintains an alpine-node container and really should be involved in this WG!

@retrohacker
Copy link
Contributor

Much ❤️! Glad to be working with you all!

@Starefossen
Copy link
Member

Awesome 😄🚀

@chorrell
Copy link

chorrell commented Aug 6, 2015

So I think we can probably just move the docker-node repo over to https://github.com/nodejs/ this week or next if everyone ls fine with that. We probably need to coordinate a few things though :)

For the short term (pre-merge) I'm fine with being the point maintainer of the docker-node image for any updates. I have admin rights to initiate the org switch, I'll just need someone to accept it on the nodejs end.

In terms of the docker-registry, we'll need to do a follow-up pull request to https://github.com/docker-library/official-images. I can handle that.

@Starefossen
Copy link
Member

Sounds like a good plan @chorrell 😄

Are there any points in merging the two repos?

@chorrell
Copy link

chorrell commented Aug 6, 2015

There are a few things I think we'll want to talk about:

Oh, also, I'm looking into dropping the 0.8 tag: nodejs/docker-node#34. v0.8.x hasn't been updated in a while, so it seems like an easy call to make. I'd like to wait until after the org switch though.

@retrohacker
Copy link
Contributor

@chorrell re: testing. I've built https://github.com/wblankenship/dante to automate testing.

I created a branch https://github.com/wblankenship/docker-iojs/tree/tests that shows dante smoke testing iojs.

Really the only work left is getting it into a Jenkins buiild. I've setup a server for us on Digital Ocean, and started writing an ansible script to automate setting it up, but am only about half way there.

@retrohacker
Copy link
Contributor

Noite: to run the tests, simply run dante in the root of that project. It reads inventory.ini and builds the dockerfiles and tests.

At this point, all we need to do is get it into Jenkins. Jenkins is pretty foreign to me, so if someone knows there way around it and wants to tackle it, here is the current state of our jenkins build server: https://github.com/wblankenship/docker-integration

@chorrell
Copy link

Oh, interesting. Nice.

Er, how do you install dante?

@retrohacker
Copy link
Contributor

Good question. Right now I'm just building from source (go install github.com/wblankenship/dante).

Relatively new to go package distribution, I assume best practice here is to deliver a prebuilt binary? I'll cut a release now.

@retrohacker
Copy link
Contributor

@chorrell woah, go's crosscompiling support is wicked awesome. Just cut a release, let me know if you have any problems with these binaries: https://github.com/wblankenship/dante/releases/tag/v1.0.0

@chorrell
Copy link

oh, nice! The dante-darwin-amd64 binary worked just fine for me

@retrohacker
Copy link
Contributor

@chorrell nodejs/build#168

@chorrell
Copy link

👍

@chorrell
Copy link

chorrell commented Sep 4, 2015

Hi all,

I'm going to kick off the org switch today to move docker-node to the nodejs org.

After that I'm going to create a fork and work on testing a Node.js 4.0.0 RC-1 docker image to prep for a 4.0.0 release.

@chorrell
Copy link

chorrell commented Sep 4, 2015

Oh, and I'll handle the update and pull request for https://github.com/docker-library/official-images too :)

@chorrell
Copy link

chorrell commented Sep 4, 2015

"You don’t have admin rights to nodejs" Doh. That didn't work the way I expected it too...

Reading this: https://help.github.com/articles/transferring-a-repository/

On the other end, the person transferring the repository also needs to have admin rights in the receiving organization.

Apparently I need admin rights in nodejs...

@mikeal
Copy link

mikeal commented Sep 4, 2015

@chorrell can you add me to that repo? I can move the repo and add whatever teams you need on it.

@chorrell
Copy link

chorrell commented Sep 4, 2015

@mikeal Will do, thanks!

@chorrell
Copy link

chorrell commented Sep 4, 2015

Done, you should be an admin for the repo now

@mikeal
Copy link

mikeal commented Sep 4, 2015

I must be pulling up the wrong link, which repo am I transferring? :)

@chorrell
Copy link

chorrell commented Sep 4, 2015

@mikeal
Copy link

mikeal commented Sep 4, 2015

transferred, i added build, docker and docker-admin teams to it.

@chorrell
Copy link

chorrell commented Sep 4, 2015

Nice, thanks!

@Starefossen
Copy link
Member

Thanks @mikeal

@chorrell
Copy link

chorrell commented Sep 4, 2015

@mikeal Hmm, I don't think the docker team has admin-type access yet: https://github.com/orgs/nodejs/teams/docker/repositories

@chorrell
Copy link

chorrell commented Sep 4, 2015

Or I just don't understand how teams and admin access etc work...

@rvagg
Copy link
Member Author

rvagg commented Sep 5, 2015

I added docker-node to the repos list for that team, you should all now have admin on both repos

screen shot 2015-09-05 at 11 29 10 am

@Starefossen
Copy link
Member

@chorrell should we consolidate the docker repos or just discontinue the docker-iojs one?

@chorrell
Copy link

chorrell commented Sep 8, 2015

@Starefossen I think we should definitely consolidate some things. For instance I would love to the automated testing work added to the nodejs image: #46

@chorrell
Copy link

chorrell commented Sep 9, 2015

Oh, and the README!

@Starefossen
Copy link
Member

I'm closing this now since the converge has been completed.

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

No branches or pull requests

5 participants