Skip to content

Commit

Permalink
removing use of semaphore cache as the public semaphore will not have…
Browse files Browse the repository at this point in the history
… cache (#145)

removing use of semaphore cache as the public semaphore will not have cache

* utilize workflow level caching to publish the built
artifacts to the tests. otherwise turn off all caching of .m2 etc

* remove .m2/settings.xml to ensure build passes without internal artifact store

---------

Co-authored-by: Xavier Léauté <xavier@confluent.io>
  • Loading branch information
jkuhnashconfluent and xvrl committed Jul 21, 2023
1 parent 23956f7 commit 3b8324d
Showing 1 changed file with 14 additions and 4 deletions.
18 changes: 14 additions & 4 deletions .semaphore/semaphore.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,8 @@ name: Apache Druid
agent:
machine:
type: s1-prod-ubuntu20-04-amd64-1

execution_time_limit:
hours: 3
blocks:
- name: "Install"
task:
Expand Down Expand Up @@ -33,11 +34,16 @@ blocks:
jobs:
- name: "Install"
commands:
- cache restore
# This is a change meant to validate semaphore public builds
# so thus removing configurations for Confluent's internal CodeArtifact
- rm ~/.m2/settings.xml
- >
MAVEN_OPTS="${MAVEN_OPTS} -Xmx3000m" ${MVN} clean install
-q -ff ${MAVEN_SKIP} ${MAVEN_SKIP_TESTS} -T1C
- cache store
# downstream tests depend on artifacts installed by mvn install into .m2
# also cache target to avoid the cost of recompiling tests
- tar zcf cache-post-install.tgz .m2 target
- artifact push workflow cache-post-install.tgz

- name: "Tests"
task:
Expand All @@ -47,7 +53,11 @@ blocks:
- echo $SEMAPHORE_WORKFLOW_ID
- sem-version java 17
- checkout
- cache restore
- artifact pull workflow cache-post-install.tgz
- tar zxf cache-post-install.tgz
# This is a change meant to validate semaphore public builds
# so thus removing configurations for Confluent's internal CodeArtifact
- rm ~/.m2/settings.xml
jobs:
- name: "animal sniffer checks"
commands:
Expand Down

0 comments on commit 3b8324d

Please sign in to comment.