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

Orb version 7.2.0 and 7.3.0 have not been published 😱 #175

Closed
fotos opened this issue Aug 13, 2023 · 26 comments
Closed

Orb version 7.2.0 and 7.3.0 have not been published 😱 #175

fotos opened this issue Aug 13, 2023 · 26 comments
Labels
bug Something isn't working

Comments

@fotos
Copy link
Collaborator

fotos commented Aug 13, 2023

Orb version

7.2.0 and 7.3.0

What happened

The versions are missing from the orb registry. The latest published version is 7.1.1.

Additionally the last 2 releases are missing from GitHub releases.

The CircleCI pipeline failed with an error:

[9:55:24 AM] [semantic-release] › ✖  Failed step "publish" of plugin "semantic-release-circleci-orb"
[9:55:24 AM] [semantic-release] › ✖  An error occurred while running semantic-release: Error: Command failed: /home/circleci/circleci orb publish packed-orb.yml react-native-community/react-native@7.2.0 --token [secure]
Error: AUTHORIZATION_FAILURE

Looks like the token is incorrect. A long time has passed since the last time the orb was published and I'm suspecting the token has been removed due to the CircleCI security incident back in December 2022 / January 2023.

This probably requires manually (re-)running the CI steps to publish the orb and create the GH releases.

Expected behavior

I'd like to use version 7.2.0, which makes the detox setup optional, in our project.

@fotos fotos added the bug Something isn't working label Aug 13, 2023
@fotos
Copy link
Collaborator Author

fotos commented Aug 13, 2023

cc: @cortinico and @devnev who merged the last couple of PRs. 🙏

@cortinico
Copy link
Member

I don't have rights to publish this orb, maybe @devnev has

@devnev
Copy link
Contributor

devnev commented Aug 14, 2023

Nope, I'm just a contributor. @matt-oakes seems to have set it up originally?

@fotos
Copy link
Collaborator Author

fotos commented Sep 18, 2023

Hi @matt-oakes! 👋

Is there anything I can do to help publish the missing orbs? Perhaps you can share the access with other maintainers to get this going?

@bhandanyan-nomad
Copy link
Contributor

I'm trying to take advantage of the change introduced in this PR #168 but it seems that it's only available in 7.3.0, which per this issue is not released.

@matt-oakes @devnev @cortinico if you could escalate to the right person to get this fixed it would be greatly appreciated 🙏

@cortinico cortinico reopened this Oct 6, 2023
@cortinico
Copy link
Member

I've sent an email to @matt-oakes as I believe he has the access to the orb publishing

@fotos
Copy link
Collaborator Author

fotos commented Oct 6, 2023

Thank you @cortinico! 🙏

@bhandanyan-nomad
Copy link
Contributor

Any update here? If @matt-oakes can't be brought in to fix the issue, what else can be done?

For now, I'm working around my issue by pulling in the android docker image with my desired version manually without the rn/linux_android executor. It's a fine solution, but would still be nice to use what this orb provides.

Thanks 🙏

@matt-oakes
Copy link
Member

Apologies for the lack of reply here. I don't routinely monitor this repo anymore as I don't use Circle CI.

I have replied to @cortinico's email and given them admin access to this repo. Additionally, there are already over 100 people with write access to this repo through the "react-native-community" members list.

It's not clear to me how to grant permission to publish on Circle CI, but @cortinico thinks that write/admin access to this Github repo will be enough. If it's not, then let me know and I can make the changes needed.

Again, apologies for the lack of reply.

@fotos
Copy link
Collaborator Author

fotos commented Oct 13, 2023

Thank you @matt-oakes. I believe admin access is required in order to access the "Security" section, in the "Settings" tab, and rotate (recreate) the CircleCI deploy keys.

@cortinico let me know if you need help with anything. For example, I believe, to re-publish the currently unpublished releases, we need to remove the current tags , and re-trigger the CI/CD that will create the GH release and publish the orbs.

@cortinico
Copy link
Member

cortinico commented Oct 13, 2023

@cortinico let me know if you need help with anything. For example, I believe, to re-publish the currently unpublished releases, we need to remove the current tags , and re-trigger the CI/CD that will create the GH release and publish the orbs.

@fotos Would you be up for fixing the Orb Development kit I started doing here?

There should be some lint issues to fix and so on. Just send a similar PR and once the PR is green we can merge it and release a new version.

@fotos
Copy link
Collaborator Author

fotos commented Oct 13, 2023

@cortinico sure, I can try.

I see that you did something similar to circle orb init in the repo. I think I can take it from here.

@fotos
Copy link
Collaborator Author

fotos commented Oct 13, 2023

@cortinico as suspected, I don't think the current setup runs builds on forks: #178

(similarly with other PRs from forks)

Unfortunately I don't have write (push) access to the repo so I can't push changes directly in your branch:

➜  react-native-circleci-orb git:(nc/update-orb-development-kit) ✗ git push upstream nc/update-orb-development-kit
ERROR: Permission to react-native-community/react-native-circleci-orb.git denied to fotos.
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

I'm not sure I want to setup all the python tools locally (or in docker) to get this going. I think it's gonna be easier to get write access so I can push directly to the branch and iterate. 🙏

@cortinico
Copy link
Member

Unfortunately I don't have write (push) access to the repo so I can't push changes directly in your branch:

I've sent you an invite. Could you force push on the branch and make sure the CI passes so we can merge this?

@fotos
Copy link
Collaborator Author

fotos commented Oct 27, 2023

Accepted the invite.

Will try to take a look over the weekend. Otherwise I’ll work on it during normal business hours next week.

@kevindice
Copy link

Thanks all - I too am keen on something available in 7.2.0.

@kevindice
Copy link

kevindice commented Dec 19, 2023

Hi @fotos, those of us using this orb are now seeing a fatal error in the Configure Detox Environment step which says that brew tap is not necessary to run anymore:

==> Tapping wix/brew
Cloning into '/opt/homebrew/Library/Taps/wix/homebrew-brew'...
remote: Enumerating objects: 747, done.        
remote: Counting objects: 100% (203/203), done.        
remote: Compressing objects: 100% (94/94), done.        
remote: Total 747 (delta 100), reused 188 (delta 91), pack-reused 544        
Receiving objects: 100% (747/747), 1.60 MiB | 2.44 MiB/s, done.
Resolving deltas: 100% (387/387), done.
Tapped 1 cask and 2 formulae (17 files, 1.6MB).
Error: Tapping homebrew/cask is no longer typically necessary.
Add --force if you are sure you need it done.

Exited with code exit status 1

In my case, I don't want to run this Detox config step at all. We requested this to be optional in #167 and @devnev and @cortinico got #135 merged this summer. It is included in the 7.2.0 version tag. Other improvements are in 7.3.0.

Without a fix to the publication pipeline for this package, we will be stuck on 7.1.1 indefinitely, and today I notice that 7.1.1 is now broken by this brew tap error.

Are you able to resolve the pipeline to get these versions published to Circle? Or do you know who can?

(I'm more than happy to help w/ GH actions config or Circle job config related to this. Seems to just be an access/permissions issue.)

@bhandanyan-nomad
Copy link
Contributor

@fotos @cortinico @matt-oakes what's the latest on this issue? Can we expect it to be resolved anytime soon?

@cortinico
Copy link
Member

The latest is that I'm looking for someone to help setting up the publishing of the orb. I have @fotos write access to the repo be he never followed up. If someone else is up for doing it, please raise your hand

@fotos
Copy link
Collaborator Author

fotos commented Jan 15, 2024

Hey folks!

I was out on extended PTO for 3 weeks and now I'm back. I'll look at the issue again now.

Personally I'd rather avoid coupling the publishing of 7.2.0 and 7.3.0 to #177 (updating the development kit). If we can get 7.2.0 and 7.3.0 published by, for example, running the pipeline manually, that would be great. Then we could update the development kit and do another release.

In any case I'm gonna have a look now. If the upgrade is mandatory I'll try to open a PR by the end of the week.

We also need the improvements from 7.2.0 and that's why I opened this issue.

@fotos
Copy link
Collaborator Author

fotos commented Jan 15, 2024

There are 2 environment variables in the CircleCI configuration for react-native-circleci-orb:

env-variables

My recommendation is to rotate the GH_TOKEN token plus the CIRCLECI_API_TOKEN (which is used by https://github.com/matt-oakes/semantic-release-circleci-orb to publish the orb), and (I believe) that should be sufficient to publish new versions in the future.

For releasing 7.2.0 and 7.3.0, after getting new values for the above environment variables, we can checkout the corresponding tags then manually edit .releaserc and trigger a yarn semantic-release to publish the orb and create the GitHub releases (and skip the CHANGELOG which is already updated).

@cortinico I'm not sure why we have to update the SDK first – am I missing something?

I would gladly do the above, if I had permissions to create the tokens, which is going to unblock everyone. Then I can work to upgrade the CircleCI SDK so we can use the SDK publishing process instead of the manual publishing process, independently and without pressure.

What do you think? Can I get ownership here and do the right thing?

@cortinico
Copy link
Member

Personally I'd rather avoid coupling the publishing of 7.2.0 and 7.3.0 to #177 (updating the development kit). If we can get 7.2.0 and 7.3.0 published by, for example, running the pipeline manually, that would be great. Then we could update the development kit and do another release.

I'm not too keen on releasing 7.2.0 and 7.3.0. IMHO we can just publish a newer version from main.

@cortinico I'm not sure why we have to update the SDK first – am I missing something?

The problem here is the CIRCLECI_API_TOKEN which we don't have practically (and I would refrain from using a personal token at this stage).
The new Orb guide is not requiring any API token at all (see https://circleci.com/docs/creating-orbs/) hence that's the reason why I think we should just update and release 7.4.0 (or 8.0.0) from main

@fotos
Copy link
Collaborator Author

fotos commented Jan 30, 2024

Personally I'd rather avoid coupling the publishing of 7.2.0 and 7.3.0 to #177 (updating the development kit). If we can get 7.2.0 and 7.3.0 published by, for example, running the pipeline manually, that would be great. Then we could update the development kit and do another release.

I'm not too keen on releasing 7.2.0 and 7.3.0. IMHO we can just publish a newer version from main.

Fair enough.

(We need to rename master to main – I might get this done after we get a new version out.)

@cortinico I'm not sure why we have to update the SDK first – am I missing something?

The problem here is the CIRCLECI_API_TOKEN which we don't have practically (and I would refrain from using a personal token at this stage). The new Orb guide is not requiring any API token at all (see https://circleci.com/docs/creating-orbs/) hence that's the reason why I think we should just update and release 7.4.0 (or 8.0.0) from main

Cool. I put a bit of effort to understand how the new SDK works (template, flow, scripts etc.) and got PR #177 working, including passing rudimentary linting / tests. I updated the PR description with more info. When you get a chance have look and let's continue the discussion in the PR.

We are close to getting this issue resolved and having a new version published. 🎉

@cortinico
Copy link
Member

(We need to rename master to main – I might get this done after we get a new version out.)

We can do this whenever needed 👍

@fotos
Copy link
Collaborator Author

fotos commented Feb 3, 2024

Great news, versions 7.2.0 and 7.3.0 are now published and available! 🎉

I'm gonna continue working on getting the CI/CD pipeline working so we can merge pending PRs and release a subsequent version too.

(If you are interested in the details of the whole saga they are available in #182)

@fotos fotos closed this as completed Feb 3, 2024
@fotos
Copy link
Collaborator Author

fotos commented Feb 3, 2024

I got the GH releases published too! 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

6 participants