Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Action to register Redhat key#35

Merged
gaborcsardi merged 1 commit intor-hub:mainfrom
jeroen:main
Jun 20, 2025
Merged

Action to register Redhat key#35
gaborcsardi merged 1 commit intor-hub:mainfrom
jeroen:main

Conversation

@jeroen
Copy link
Member

@jeroen jeroen commented Jun 17, 2025

Simple wrapper for the Redhat subscription-manager command to automatically unregister the key in a post step.

This unregisters the key in the post step, even when R CMD check hangs.

Example usage:

      - uses: r-hub/actions/register-redhat@v1
        env:
          REDHAT_ORG: ${{ secrets.REDHAT_ORG }}
          REDHAT_KEY: ${{ secrets.REDHAT_KEY }}

- uses: r-hub/actions/register-redhat@v1
env:
REDHAT_ORG: ${{ secrets.REDHAT_ORG }}
REDHAT_KEY: ${{ secrets.REDHAT_KEY }}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any reason why these are not input parameters of the action?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just to keep it super simple. Parsing inputs requires importing "@actions/core" and compiling the module I think? Right now it is just a few lines of vanilla JS...

Copy link
Contributor

@gaborcsardi gaborcsardi Jun 20, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But then why not a shell script, so it does not need node?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

composite actions do not support the post step, which is the main purpose of this action, to guarantee that the unregistration always happens.

@gaborcsardi gaborcsardi merged commit b75ada6 into r-hub:main Jun 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants