updating artifactory link to point to new artifactory #892

Merged
merged 1 commit into from Jun 9, 2017

Conversation

Projects
None yet
4 participants
Contributor

lbergelson commented Jun 8, 2017

  • the Broad artifactory moved from a self hosted repository at artifactory.broadinstitute.org
    to a web based artifactory at broadinstitute.jfrog.io/broadinstitute/
    this was causing our snapshot uploads to fail
  • updating our snapshot repository to point to the new link
  • the old artifactory has a redirect in place to the new artifactory so
    artifacts relying on the old url should still resolve. For unknown
    reasons this redirect doesn't work with gradle uploads, probably a
    gradle bug.
@lbergelson lbergelson updating artifactory link to point to new artifactory
* the Broad artifactory moved from a self hosted repository at artifactory.broadinstitute.org
to a web based artifactory at broadinstitute.jfrog.io/broadinstitute/
this was causing our snapshot uploads to fail
* updating our snapshot repository to point to the new link
* the old artifactory has a redirect in place to the new artifactory so
artifacts relying on the old url should still resolve.  For unknown
reasons this redirect doesn't work with gradle uploads, probably a
gradle bug.
0adadc7

lbergelson referenced this pull request Jun 8, 2017

Closed

Request: Release 2.10.1 #891

Contributor

magicDGS commented Jun 8, 2017

One question about this for downstream projects using HTSJDK/GATK/etc: will the old artifactory links continue working forever or should we change them in the near future? Thanks in advance!

Contributor

lbergelson commented Jun 8, 2017

@magicDGS I was given a promise to keep the link up for at least a year, but I'm going to try to advocate for keeping the link around forever because it seems like really bad form to break old builds that way.

It's probably prudent to update even if the old link stays around though.

Codecov Report

Merging #892 into master will increase coverage by 0.136%.
The diff coverage is n/a.

@@              Coverage Diff               @@
##              master     #892       +/-   ##
==============================================
+ Coverage     65.075%   65.21%   +0.136%     
- Complexity      7251     7346       +95     
==============================================
  Files            528      528               
  Lines          31931    32320      +389     
  Branches        5452     5577      +125     
==============================================
+ Hits           20779    21076      +297     
- Misses          9009     9089       +80     
- Partials        2143     2155       +12
Impacted Files Coverage Δ Complexity Δ
src/main/java/htsjdk/samtools/SAMRecordUtil.java 23.529% <0%> (-20.915%) 2% <0%> (ø)
src/main/java/htsjdk/samtools/SAMRecord.java 66.176% <0%> (+0.245%) 286% <0%> (+1%) ⬆️
...dk/variant/variantcontext/GenotypeLikelihoods.java 85.294% <0%> (+0.632%) 85% <0%> (+16%) ⬆️
...a/htsjdk/samtools/cram/encoding/rans/Decoding.java 85.714% <0%> (+3.571%) 12% <0%> (+1%) ⬆️
src/main/java/htsjdk/samtools/SAMUtils.java 62.957% <0%> (+3.574%) 173% <0%> (+51%) ⬆️
src/main/java/htsjdk/variant/vcf/VCFEncoder.java 92.05% <0%> (+3.69%) 96% <0%> (+26%) ⬆️
src/main/java/htsjdk/tribble/util/HTTPHelper.java 55.738% <0%> (+4.627%) 4% <0%> (ø) ⬇️

lbergelson requested a review from droazen Jun 9, 2017

Contributor

lbergelson commented Jun 9, 2017

@droazen Simple change here.

@droazen

droazen approved these changes Jun 9, 2017

👍 merge when ready

lbergelson was assigned by droazen Jun 9, 2017

@lbergelson lbergelson merged commit 4626fe6 into master Jun 9, 2017

4 of 5 checks passed

codecov/changes 4 files have unexpected coverage changes not visible in diff.
Details
codecov/patch Coverage not affected when comparing 98c2438...0adadc7
Details
codecov/project 65.21% (+0.136%) compared to 98c2438
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

lbergelson deleted the lb_fix_artifactory_link branch Jun 9, 2017

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