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

Check-in Jenkins Docker ORT CI #1411

Merged
merged 119 commits into from
Oct 4, 2022
Merged

Conversation

jessemcfarland
Copy link

PR Checklist

  • This PR is up-to-date with the top of all sub-component repositories except for those sub-components which are the subject of this PR. Please consult the ufs-weather-model wiki if you are unsure how to do this.

  • This PR has been tested using a branch which is up-to-date with the top of all sub-component repositories except for those sub-components which are the subject of this PR

  • An Issue describing the work contained in this PR has been created either in the subcomponent(s) or in the ufs-weather-model. The Issue should be created in the repository that is most relevant to the changes in contained in the PR. The Issue and the dependent sub-component PR
    are specified below.

  • Results for one or more of the regression tests change and the reasons for the changes are understood and explained below.

  • New or updated input data is required by this PR. If checked, please work with the code managers to update input data sets on all platforms.

Description

This PR adds a Jenkins pipeline to this repository that enables the Docker ORT CI workflow on the EPIC Jenkins server.

Testing

To test the pipeline, a new label, jenkins-ci must be created and added to this PR. Subsequent builds can be triggered by adding a comment with the string REBUILD.

@jkbk2004 jkbk2004 added jenkins-ci Jenkins CI: ORT build/test on docker container and removed jenkins-ci Jenkins CI: ORT build/test on docker container labels Oct 4, 2022
@BrianCurtis-NOAA
Copy link
Collaborator

@jkbk2004 is this ready to go now?

@jkbk2004
Copy link
Collaborator

jkbk2004 commented Oct 4, 2022

@jkbk2004 is this ready to go now?
I was about to add reviewers. We can start reviews after seeing ORT run.

@BrianCurtis-NOAA
Copy link
Collaborator

Just noting: If there is supposed to be details in the ORT task, it comes back with a 404 error.

@flampouris
Copy link

@jessemcfarland please communicate the log file to the appropriate members.

@jkbk2004
Copy link
Collaborator

jkbk2004 commented Oct 4, 2022

@jessemcfarland can you attach the log file output to this PR conversation? so that reviewers can take a look.

@BrianCurtis-NOAA
Copy link
Collaborator

We can merge this in as there is no change to the running of the model/physics etc.

What I would encourage is that the details for Jenkins tasks be made available to those in the public as this whole project is a community one. If this is possible, but not for a while, then it would be good to allow me to see the logs as I am responsible for operational targets of the UFSWM.

@jessemcfarland
Copy link
Author

We can merge this in as there is no change to the running of the model/physics etc.

What I would encourage is that the details for Jenkins tasks be made available to those in the public as this whole project is a community one. If this is possible, but not for a while, then it would be good to allow me to see the logs as I am responsible for operational targets of the UFSWM.

@BrianCurtis-NOAA I'm working on gathering the logs now. I'll attach them to the PR shortly. We're still discussing the access model for Jenkins including access to the logs for PRs.

@jessemcfarland
Copy link
Author

Here are the Jenkins logs for the last pipeline run: ufs-weather-model_PR-1411_jenkins_logs.zip

@BrianCurtis-NOAA
Copy link
Collaborator

@jkbk2004 Should the label be removed once the job has started?

@jkbk2004 jkbk2004 removed the jenkins-ci Jenkins CI: ORT build/test on docker container label Oct 4, 2022
@jkbk2004
Copy link
Collaborator

jkbk2004 commented Oct 4, 2022

@BrianCurtis-NOAA Best option is to run job when the label is added. Once jobs runs, jenkins will remove the label. And iterative steps ... Will come with a final solution for triggering mechanism.

@jkbk2004 jkbk2004 merged commit e6da626 into ufs-community:develop Oct 4, 2022
@jkbk2004 jkbk2004 deleted the jenkins-ci branch November 4, 2022 12:52
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.

4 participants