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

Run e3sm gh action only if repo name is e3sm #2758

Merged
merged 1 commit into from
Mar 14, 2024

Conversation

bartgol
Copy link
Contributor

@bartgol bartgol commented Mar 13, 2024

Ideally, if a gh action file starts with e3sm-, then it should only run on the e3sm repo (and have a check inside that that's the case). On top of that, the singularity tests seem to have little meaning for the SCREAM fork, since we virtually never modify code outside the eamxx folder, and in the rare instances where we do, it's probably in EAM, which does get tested via SCREAMv0 label.

@bartgol bartgol added AT: AUTOMERGE Inform the autotester (AT) that it can merge this PR if reviewers approved, and tests pass AT: Integrate Without Testing labels Mar 13, 2024
@bartgol bartgol requested review from jgfouca and mahf708 March 13, 2024 20:29
@bartgol bartgol self-assigned this Mar 13, 2024
@E3SM-Autotester
Copy link
Collaborator

Status Flag 'Pull Request AutoTester' - Testing Jenkins Projects:

Pull Request Auto Testing STARTING (click to expand)

Build Information

Test Name: SCREAM_PullRequest_Autotester_Mappy

  • Build Num: -1
  • Status: SKIPPED

Jenkins Parameters

Parameter Name Value

Build Information

Test Name: SCREAM_PullRequest_Autotester_Weaver

  • Build Num: -1
  • Status: SKIPPED

Jenkins Parameters

Parameter Name Value

Using Repos:

Repo: SCREAM (E3SM-Project/scream)
  • Branch: bartgol/gh-actions-filter
  • SHA: 3343d21
  • Mode: TEST_REPO

Pull Request Author: bartgol

@E3SM-Autotester
Copy link
Collaborator

Status Flag 'Pull Request AutoTester' - Jenkins Testing: all Jobs PASSED

Pull Request Auto Testing has PASSED (click to expand)

Build Information

Test Name: SCREAM_PullRequest_Autotester_Mappy

  • Build Num: -1
  • Status: SKIPPED

Jenkins Parameters

Parameter Name Value

Build Information

Test Name: SCREAM_PullRequest_Autotester_Weaver

  • Build Num: -1
  • Status: SKIPPED

Jenkins Parameters

Parameter Name Value

@E3SM-Autotester
Copy link
Collaborator

Status Flag 'Pre-Merge Inspection' - - This Pull Request Requires Inspection... The code must be inspected by a member of the Team before Testing/Merging
THE LAST COMMIT TO THIS PULL REQUEST HAS BEEN REVIEWED, BUT NOT ACCEPTED OR REQUIRES CHANGES!

@E3SM-Autotester
Copy link
Collaborator

All Jobs Finished; status = PASSED, However Inspection must be performed before merge can occur...

@E3SM-Autotester E3SM-Autotester merged commit a5e6f59 into master Mar 14, 2024
8 checks passed
@E3SM-Autotester E3SM-Autotester deleted the bartgol/gh-actions-filter branch March 14, 2024 04:32
@E3SM-Autotester E3SM-Autotester removed the AT: AUTOMERGE Inform the autotester (AT) that it can merge this PR if reviewers approved, and tests pass label Mar 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants