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

Conversion to Enterprise account #905

Closed
mcollina opened this issue Jul 26, 2024 · 32 comments
Closed

Conversion to Enterprise account #905

mcollina opened this issue Jul 26, 2024 · 32 comments

Comments

@mcollina
Copy link
Member

Screenshot 2024-07-26 at 09 06 07

Currently we are on the Enterprise Cloud system, and we are going to be migrated to "Enterprise Account" on September 3rd. I recommend we do it earlier, between now and then, and deal with all the possible hiccups on our terms.

Alternatively, we let the deadline expire and deal with anything in September. I feel this might be a bit disruptive for the collaborators because of the upcoming cutoff for v23, so I'd recommend we do this asap.

@targos
Copy link
Member

targos commented Jul 28, 2024

I was told in private that we should let the foundation create an enterprise account and move this org (and probably nodejs-private too) under it.

@mcollina
Copy link
Member Author

I have been told that said migration is not as straightforward as we think, so an independent org it's better. (we have quite a lot of custom stuff here).

@targos
Copy link
Member

targos commented Jul 28, 2024

Then let's do it!

@mcollina
Copy link
Member Author

@nodejs/tsc are there any objections?

@anonrig
Copy link
Member

anonrig commented Jul 28, 2024

+1 let's do it

@ovflowd
Copy link
Member

ovflowd commented Jul 28, 2024

I have been told that said migration is not as straightforward as we think, so an independent org it's better. (we have quite a lot of custom stuff here).

I don't think that was the outcome of the CPC meeting. If I recall, what Ben from the Foundation meant is that billing settings could be tricky, but to be honest, nothing that GitHub staff couldn't help us out.

IMO, if we already need to move to the GitHub Enterprise system, I don't see why it shouldn't be under the OpenJS Foundation "Enterprise." Does it make any difference? 🤔

@mcollina
Copy link
Member Author

I don't think that was the outcome of the CPC meeting. If I recall, what Ben from the Foundation meant is that billing settings could be tricky, but to be honest, nothing that GitHub staff couldn't help us out.

Given we have a wild amount of custom discount & quotas from GitHub, a problematic scenario is that the project is "broken" for some time before we try to fix this. I'm absolutely certain that GitHub staff can fix this, but why breaking things?

IMO, if we already need to move to the GitHub Enterprise system, I don't see why it shouldn't be under the OpenJS Foundation "Enterprise." Does it make any difference?

See above.

@ovflowd
Copy link
Member

ovflowd commented Jul 29, 2024

Moving to an Enterprise regardless of a Node.js one or OpenJS one could affect billing.

We have a scheduled call with GitHub staff to clear out questions. Isn't it worth to first clarify things rather than move by fear?

@mcollina
Copy link
Member Author

We have a scheduled call with GitHub staff to clear out questions. Isn't it worth to first clarify things rather than move by fear?

I would prefer not to change anything at all. They are the ones forcing a change.


we should also apply this to nodejs-private.

@ovflowd
Copy link
Member

ovflowd commented Jul 29, 2024

Well they, as, GitHub. So if they are forcing a change, shouldn't we ask counselling to them regardless? To check if anything would break at all, regardless of OpenJS Enterprise?

@targos
Copy link
Member

targos commented Jul 29, 2024

we should also apply this to nodejs-private.

I would like to try and put nodejs-private under the same enterprise account. It could have the nice side-effect to remove the current limitation we have on runner minutes.

@mcollina
Copy link
Member Author

shouldn't we ask counselling to them regardless? To check if anything would break at all, regardless of OpenJS Enterprise?

I have no time for another meeting, unfortunately.

It could have the nice side-effect to remove the current limitation we have on runner minutes.

We have the maximum amount of free credits we could get in nodejs-private already. The chances of losing that and having to re-negotiate it are high enough (free credits are indeed related to billing).


It seems apparent that we do not have consensus on this move, and I do not have the time to "push it through" and have meetings about it. Therefore, I will not take care of this.

(Hopefully somebody else will, or we do have the forced migration in September).

@ovflowd
Copy link
Member

ovflowd commented Jul 29, 2024

I'm no TSC, but I'd be more than happy to help on the OpenJS side of things as a CPC member.

@mhdawson
Copy link
Member

I also prefer to keep the organization separate, maybe once there is more experience by other projects having moved we might consider again. I see keeping it separate and transitioning to our own enterprise account as the least risk which is why I would prefer that.

@ovflowd
Copy link
Member

ovflowd commented Jul 31, 2024

I also prefer to keep the organization separate, maybe once there is more experience by other projects having moved we might consider again. I see keeping it separate and transitioning to our own enterprise account as the least risk which is why I would prefer that.

Fair enough.

@ruyadorno
Copy link
Member

+1 to move forward with the migration and keep the orgs separate

@mhdawson
Copy link
Member

@targos I wonder if we could flips the two orgs (nodejs and nodejs-private) separtely to their own Enterprise orgs to reduce risk.

We could then try adding a new private org under the Enterprise Node.js org. We could then experiment to see if that new org has unlimited minutes and if so move the repos from the nodejs-private org over to the Node.js enterprise org.

That might be more work, but let us transition as/when we have time to test out/transition.

@targos
Copy link
Member

targos commented Aug 1, 2024

We could then try adding a new private org under the Enterprise Node.js org. We could then experiment to see if that new org has unlimited minutes and if so move the repos from the nodejs-private org over to the Node.js enterprise org.

I had a look and we have the same number of included minutes in both orgs (50k/month). We use 0 on the nodejs org because no private repo uses GitHub actions.

The main difference is that the private org has billing enabled with a spending limit so we can use more minutes if necessary. This would have to be moved to the new enterprise account.

It would be much easier to administrate everything if both orgs used only one enterprise account but I won't object to creating two if you're really afraid that we might lose credits or privileges.

@mcollina
Copy link
Member Author

Done for the nodejs one. Does anybody know how to merge the private one with the main one?

@richardlau
Copy link
Member

Done for the nodejs one. Does anybody know how to merge the private one with the main one?

Possibly via "Invite organization" on https://github.com/enterprises/nodejs/organizations?

@mcollina
Copy link
Member Author

Screenshot 2024-08-14 at 17 16 56

unfortunately that does not seem to be possible.

@targos
Copy link
Member

targos commented Aug 14, 2024

@mcollina We have unlimited seats. I just added one, can you try again?

@mcollina
Copy link
Member Author

mcollina commented Aug 14, 2024

Done. However before inviting I would like to remove all non-tsc members as enterprise owners, as this access is not needed by moderation.

Alternatively, we could keep limit it
to the TSC chair, vice-chair, and openjs representative.

We should do this before adding the nodejs-private org.

@ovflowd
Copy link
Member

ovflowd commented Aug 14, 2024

Alternatively, we could keep limit it to the TSC chair, vice-chair, and openjs representative.

We should do this before adding the nodejs-private org.

This makes sense IMO

@richardlau
Copy link
Member

richardlau commented Aug 14, 2024

Our GitHub actions are now backlogged. I'm getting lots of notifications for canceled Auto Start CI runs which started appearing time-wise shortly after the conversion 😞.

@ovflowd
Copy link
Member

ovflowd commented Aug 14, 2024

That's bad, any updates on that?

@mcollina
Copy link
Member Author

mcollina commented Aug 15, 2024

I've removed from Enterprise owners:

Enterprise owners are now all TSC members + @bensternthal.

@bensternthal
Copy link

@mcollina @ovflowd thanks for the heads up.

Note one thing I noticed when I did this upgrade on other projects is that some of our CI broke (looks like there is a similar issue here). At least for me, this was caused by some stricter rules in GH enterprise (and apologies if you know this already)

Screenshot 2024-08-15 at 8 03 20 AM

I solved this by adding more granular permissions into the workflow .yml file example

Hope this helps!

@richardlau
Copy link
Member

That's bad, any updates on that?

Looks a lot better today.

https://github.com/enterprises/nodejs/settings/actions/hosted-runners yesterday was reporting a concurrency limit of 18 which has slowly been creeping upwards (in a private Slack thread it was noted later showing 23 and then 29) -- currently the concurrency limit is 131. We're hoping this will self-resolve.

@marco-ippolito
Copy link
Member

related failure #911

@mcollina
Copy link
Member Author

I've ticked to allow gh actions to create and approve PRs.

@mcollina
Copy link
Member Author

Closing as completed, feel free to reopen in case more problems arises.

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

No branches or pull requests

9 participants