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

Rename primary branch to main #35

Closed
mhdawson opened this issue Jun 17, 2021 · 7 comments
Closed

Rename primary branch to main #35

mhdawson opened this issue Jun 17, 2021 · 7 comments
Assignees

Comments

@mhdawson
Copy link
Member

We are going through all of the node.js repositories to rename the primary branch to main. Please see nodejs/node#33864 for more context.

There are some references to the master branch in the files so I think I'll need some help in doing the update. @richardlau is that something you can help with?

@richardlau richardlau self-assigned this Sep 13, 2021
@richardlau
Copy link
Member

@mhdawson I can do this.

@nschonni
Copy link
Member

I opened up nodejs/build#2831 for the hook

@mhdawson
Copy link
Member Author

@richardlau, @nschonni I think in addition to nodejs/build#2831 along with the default branch rename, we would need to log into the machine the webhook and scripts are deployed to and update the files there manually as well. Is there anything else you think we need to do?

Looking at the repo itself the only references to master seen to be in external repos: https://github.com/nodejs/unofficial-builds/search?q=master so I don't think those need to be updated.

@richardlau
Copy link
Member

@rvagg When we tried to deploy nodejs/build#2831 via Ansible to the unofficial-builds server we got stuck on the github_deploy_webhook_secret -- eventually we "cheated" by reading it off the machine from the unofficial-builds-deploy-webhook.json file. Is the secret in secrets? We couldn't find it.

@richardlau
Copy link
Member

Branch has been renamed 🎉 .

@rvagg
Copy link
Member

rvagg commented May 10, 2022

uh, I can't see it in secrets. I can see it in a local host_vars file I have but that's obviously not helpful and other than in an inventory.yml in secrets I'm not sure where else it could be. I haven't really got into the habit of using inventory.yml in secrets for anything other than jenkins tokens, using the secrets workflow for inventory stuff is all new fangled for me who's been used to just building up a host_vars directory with stuff that's not properly shared! What's the best way to get this in, can I just put it as a variable in inventory.yml (infra I suppose) and it'll work, or do I need to do something else to wire it up in the playbook to fetch it out of the secrets inventory?

@richardlau
Copy link
Member

@rvagg yes, you can put the variable in the inventory in secrets and it'll work. I'm happy to update (I'll go and do so) -- we just weren't sure if we were missing something and it was already there -- neither @mhdawson nor myself have access to the github-bot part of secrets so we weren't sure if perhaps it was in there (sounds like it isn't there at all).

For completeness sake (FYI) I did add a role to our playbooks for decrypting stuff from secrets (https://github.com/nodejs/build/tree/master/ansible/roles/read-secrets) but that was for the cases where 1) the same secret applied to multiple hosts (e.g. the RHEL subscription id for the RHEL machines) and I wanted to avoid duplicating the same thing multiple times in the inventory and/or 2) the secret is longer than a short string (e.g. the ssh key all of the release machines need to upload to the staging server).

richardlau added a commit to nodejs/build that referenced this issue May 12, 2022
Remove comment about creating local host_vars as the required
`github_deploy_webhook_secret` secret has been added to the
secrets repository.

Refs: nodejs/unofficial-builds#35 (comment)
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

4 participants