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

[DevOps] Fix Jenkins AirGap Groovy Scripting #841

Open
irishgordo opened this issue May 31, 2023 · 4 comments
Open

[DevOps] Fix Jenkins AirGap Groovy Scripting #841

irishgordo opened this issue May 31, 2023 · 4 comments
Assignees

Comments

@irishgordo
Copy link
Contributor

Pipeline is failing, during harvester-installer step where we get the repo then use the node to build the artifacts fresh (instead of gathering from some hosted endpoint)

Fix:

+ git checkout origin/master
Previous HEAD position was a71e111 testing: Test
HEAD is now at 746783b remove --kill-who flags
+ git pull upstream master
From https://github.com/harvester/harvester-installer
 * branch            master     -> FETCH_HEAD
+ git checkout feat/airgap-harvester-airgap-rancher-install-test
Previous HEAD position was 746783b remove --kill-who flags
Switched to branch 'feat/airgap-harvester-airgap-rancher-install-test'
+ git merge origin/master
Committer identity unknown

*** Please tell me who you are.

Run

  git config --global user.email "you@example.com"
  git config --global user.name "Your Name"

to set your account's default identity.
Omit --global to set the identity only in this repository.

fatal: unable to auto-detect email address (got 'jenkins@jenkins-test-ci.(none)')
Already up to date.
Your branch is up to date with 'origin/feat/airgap-harvester-airgap-rancher-install-test'.

Cross Ref:
#686
harvester/harvester-installer#400

@irishgordo irishgordo self-assigned this May 31, 2023
@irishgordo
Copy link
Contributor Author

Trying just a fix with:

jenkins@jenkins-test-ci:~/workspace$ git config --global user.email "jenkins@jenkins-test-ci.harvesterhci.io"
jenkins@jenkins-test-ci:~/workspace$ git config --global user.name "Jenkins AirGapTesting"

To act as a temporary band-aid for the time being, tbd if it works...

@irishgordo
Copy link
Contributor Author

Since jenkins is a service user on the node after setting just validated with:

jenkins@jenkins-test-ci:/home/ubuntu$ git config --get user.email
jenkins@jenkins-test-ci.harvesterhci.io
jenkins@jenkins-test-ci:/home/ubuntu$ git config --get user.name
Jenkins AirGapTesting

The pipeline is doing some work to rebase from the feature branch of harvester-installer onto the main/master branch, pretty sure that's why it's needing to have "who's writing this commit".

Ultimately, the harvester-installer fork's feature branch will get merged in upstream to hopefully solidify this as a non-issue.
But again, just as a testing band-aid right now, applying this fix manually/ad-hoc...

@irishgordo
Copy link
Contributor Author

And again we need all the Dapper/Docker/CMake/Go/Compling based things to build the artifacts since we leverage harvester-installer, therefore currently Jenkins is not containerized/pod-afied on the node because of that...

@irishgordo
Copy link
Contributor Author

Moved to automation test implement - but really this is an adhoc edit/band-aid that will need further improvement...

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

No branches or pull requests

1 participant