Skip to content

Commit 3896068

Browse files
kfarazkgyrtkirk
andauthored
Remove maven.twttr repo from pom (#17797)
remove usage of dependency:go-offline from build scripts - as it tries to download excluded artifacts --------- Co-authored-by: Zoltan Haindrich <[email protected]>
1 parent dfc20f9 commit 3896068

File tree

4 files changed

+3
-13
lines changed

4 files changed

+3
-13
lines changed

.github/workflows/cron-job-its.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -127,7 +127,7 @@ jobs:
127127
cache: maven
128128

129129
- name: maven build # needed to rebuild incase of maven snapshot resolution fails
130-
run: mvn clean install dependency:go-offline -P dist -P skip-static-checks,skip-tests -Dmaven.javadoc.skip=true -Dcyclonedx.skip=true -Dweb.console.skip=true
130+
run: mvn clean install -P dist -P skip-static-checks,skip-tests -Dmaven.javadoc.skip=true -Dcyclonedx.skip=true -Dweb.console.skip=true
131131

132132
- name: security vulnerabilities check
133133
run: |

integration-tests/script/setup_druid_on_k8s.sh

+1-1
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ cp -r client_tls docker/client_tls
2626
cd ..
2727

2828
# Build Docker images for pods
29-
mvn -B -ff -q dependency:go-offline \
29+
mvn -B -ff -q \
3030
install \
3131
-Pdist,bundle-contrib-exts \
3232
-Pskip-static-checks,skip-tests \

it.sh

+1-1
Original file line numberDiff line numberDiff line change
@@ -229,7 +229,7 @@ case $CMD in
229229
usage
230230
;;
231231
"ci" )
232-
mvn -q clean install dependency:go-offline -P dist $MAVEN_IGNORE -T1C
232+
mvn -q clean install -P dist $MAVEN_IGNORE -T1C
233233
;;
234234
"build" )
235235
mvn -B clean install -P dist $MAVEN_IGNORE -T1.0C $*

pom.xml

-10
Original file line numberDiff line numberDiff line change
@@ -293,16 +293,6 @@
293293
<enabled>false</enabled>
294294
</snapshots>
295295
</repository>
296-
297-
<!--
298-
maven-dependency-plugin:3.1.2 seems to have updated HTTP repository access behavior.
299-
We get the following error "Blocked mirror for repositories: [twitter (http://maven.twttr.com, default, releases+snapshots)]"
300-
The suggested action step is to add the mirror: https://maven.apache.org/docs/3.8.1/release-notes.html#how-to-fix-when-i-get-a-http-repository-blocked
301-
-->
302-
<repository>
303-
<id>twitter</id>
304-
<url>https://maven.twttr.com</url>
305-
</repository>
306296
</repositories>
307297

308298
<pluginRepositories>

0 commit comments

Comments
 (0)