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

Added github action that checks that we can run make and build RPM #526

Merged
merged 7 commits into from
Feb 5, 2024

Conversation

shreyb
Copy link
Collaborator

@shreyb shreyb commented Feb 5, 2024

This workflow will run when a PR is merged, or when any other push is made to the master branch.

Originally the plan was to run this workflow at PR open, but that
doesn't seem possible due to:
https://github.blog/changelog/2021-02-19-github-actions-workflows-triggered-by-dependabot-prs-will-run-with-read-only-permissions/

We need a repo secret to access fermitools/jobsub_lite_config.  Since we
can't do that, for now, at least run after a PR merge or any other push
to master.
@shreyb shreyb marked this pull request as draft February 5, 2024 22:01
@shreyb shreyb marked this pull request as ready for review February 5, 2024 22:11
@marcmengel marcmengel merged commit fc0d742 into fermitools:master Feb 5, 2024
2 checks passed
@shreyb shreyb added this to the 1.6 milestone Feb 5, 2024
marcmengel pushed a commit that referenced this pull request Feb 16, 2024
)

* Added github action that checks that we can run make and build RPM

* Added pull_request to events that trigger this workflow

* Added build status badge for build-al9 workflow

* Run workflow at push and merged PR

Originally the plan was to run this workflow at PR open, but that
doesn't seem possible due to:
https://github.blog/changelog/2021-02-19-github-actions-workflows-triggered-by-dependabot-prs-will-run-with-read-only-permissions/

We need a repo secret to access fermitools/jobsub_lite_config.  Since we
can't do that, for now, at least run after a PR merge or any other push
to master.

* Made build steps into custom action

* Move container prep to its own action

* Revert "Move container prep to its own action"

This reverts commit fd091ff.
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