osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/31054 )
Change subject: jobs/master,gerrit: fix pysim, sysmo-usim-tool
......................................................................
Patch Set 1:
This change is ready for review.
--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/31054
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I339d9ba4ad0c959d7325820eb53bfa1f0e04c164
Gerrit-Change-Number: 31054
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Comment-Date: Tue, 24 Jan 2023 14:46:20 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: osmith.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/31053 )
Change subject: jobs/master,gerrit: add docker_run_ccache
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
what do we win with this? All I see is having to maintain 2 differnet sets of commands now.
--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/31053
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: Ib6d7f309b972979d3a3bc87f8fcf71f9e930c0da
Gerrit-Change-Number: 31053
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 24 Jan 2023 14:43:58 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ci/+/31054 )
Change subject: jobs/master,gerrit: fix pysim, sysmo-usim-tool
......................................................................
jobs/master,gerrit: fix pysim, sysmo-usim-tool
Don't attempt to create the CCACHE_DIR and chown it to osmocom-build for
the simtester jenkins node, and more generally all that were not
created with the ansible provisioning scripts. In case of simtester,
there is no osmocom-build user and we don't have the space there to use
ccache.
Use a new environment variable PROVISIONED_WITH_ANSIBLE that I've added
in the jenkins configuration to all nodes that were provisioned with
ansible.
Fix for:
chown: invalid user: ‘osmocom-build:osmocom-build’
Related: OS#5848
Change-Id: I339d9ba4ad0c959d7325820eb53bfa1f0e04c164
---
M jobs/gerrit-verifications.yml
M jobs/master-builds.yml
2 files changed, 10 insertions(+), 6 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ci refs/changes/54/31054/1
diff --git a/jobs/gerrit-verifications.yml b/jobs/gerrit-verifications.yml
index 66077d6..dca5e05 100644
--- a/jobs/gerrit-verifications.yml
+++ b/jobs/gerrit-verifications.yml
@@ -696,9 +696,11 @@
builders:
- shell: |-
- CCACHE_DIR="$HOME/ccache/gerrit-verifications"
- mkdir -p "$CCACHE_DIR"
- chown osmocom-build:osmocom-build "$CCACHE_DIR"
+ if [ "$PROVISIONED_WITH_ANSIBLE" = 1 ]; then
+ CCACHE_DIR="$HOME/ccache/gerrit-verifications"
+ mkdir -p "$CCACHE_DIR"
+ chown osmocom-build:osmocom-build "$CCACHE_DIR"
+ fi
- shell: '{obj:cmd}'
publishers:
diff --git a/jobs/master-builds.yml b/jobs/master-builds.yml
index a690fa9..f601c4f 100644
--- a/jobs/master-builds.yml
+++ b/jobs/master-builds.yml
@@ -567,9 +567,11 @@
builders:
- shell: |-
- CCACHE_DIR="$HOME/ccache/master-builds"
- mkdir -p "$CCACHE_DIR"
- chown osmocom-build:osmocom-build "$CCACHE_DIR"
+ if [ "$PROVISIONED_WITH_ANSIBLE" = 1 ]; then
+ CCACHE_DIR="$HOME/ccache/master-builds"
+ mkdir -p "$CCACHE_DIR"
+ chown osmocom-build:osmocom-build "$CCACHE_DIR"
+ fi
- shell: '{obj:cmd}'
publishers:
--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/31054
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I339d9ba4ad0c959d7325820eb53bfa1f0e04c164
Gerrit-Change-Number: 31054
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-MessageType: newchange
osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/pysim/+/31052 )
Change subject: Do not merge: test commit for testing jenkins
......................................................................
Do not merge: test commit for testing jenkins
Change-Id: Ifb1f03b06062e3f6eb6738c6ada8f24e6d01a219
---
A test
1 file changed, 0 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/52/31052/1
diff --git a/test b/test
new file mode 100644
index 0000000..e69de29
--- /dev/null
+++ b/test
--
To view, visit https://gerrit.osmocom.org/c/pysim/+/31052
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: Ifb1f03b06062e3f6eb6738c6ada8f24e6d01a219
Gerrit-Change-Number: 31052
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-MessageType: newchange