Skip to content
This repository was archived by the owner on Feb 21, 2023. It is now read-only.

Adds jenkins build/publish/pipeline scripts to utils repo#23

Closed
IgorTodorovskiIBM wants to merge 5 commits intozopencommunity:mainfrom
IgorTodorovskiIBM:jenkins
Closed

Adds jenkins build/publish/pipeline scripts to utils repo#23
IgorTodorovskiIBM wants to merge 5 commits intozopencommunity:mainfrom
IgorTodorovskiIBM:jenkins

Conversation

@IgorTodorovskiIBM
Copy link
Copy Markdown
Member

  • The pipeline.jenkins scripts drive the Jenkins pipeline, which calls the Build and Promote job
  • The build.jenkins script builds a utils compliant project and generates a pax.Z artifact
  • The publish.jenkins script publishes a pax.Z to as a github release

https://128.168.139.253:8443/view/Framework/ has been updated to call these scripts

Comment thread cicd/build.jenkins
@@ -0,0 +1,33 @@
# Assumes utils repo is cloned into PWD
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is 'groovy' code correct?

Comment thread cicd/build.jenkins
export PATH="$PWD/bin:$PATH"

# Get port name based on git repo
PORT_NAME=$(basename "${PORT_GITHUB_URL}")
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

shouldn't this be PORT_GIT_URL and not PORT_GITHUB_URL ?

Comment thread cicd/build.jenkins
@@ -0,0 +1,33 @@
# Assumes utils repo is cloned into PWD

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

can you add a description? I'm not sure which env vars are set in /jenkins/.env and which (if any) are set before-hand? This appears to also only work for a GIT port and not a TARBALL port.

Comment thread cicd/build.jenkins
export PORT_INSTALL_DIR="${PWD}/install"
mkdir -p ${PORT_INSTALL_DIR}

git clone -b "${BRANCH}" "${PORT_GITHUB_URL}" ${PORT_NAME} && cd ${PORT_NAME}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

how is BRANCH related to the PORT_GIT_BRANCH ?

Comment thread cicd/pipeline.jenkins
@@ -0,0 +1,18 @@
def repo = params.get("REPO")
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

As with the other jenkins files, a description would be good. Do we want to have some sort of correlation between 'sh' env vars and these jenkins vars? Should they be consistently named?

@IgorTodorovskiIBM
Copy link
Copy Markdown
Member Author

Closing in favour of zopencommunity/meta#11

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants