Skip to content

travis: use https based maven repo mirror#3845

Merged
yadvr merged 1 commit into
apache:4.13from
shapeblue:travis-maven-https-fix
Jan 28, 2020
Merged

travis: use https based maven repo mirror#3845
yadvr merged 1 commit into
apache:4.13from
shapeblue:travis-maven-https-fix

Conversation

@yadvr
Copy link
Copy Markdown
Member

@yadvr yadvr commented Jan 28, 2020

Travis has been failing lately as it requires that repositories use
https:// URLs.

Reference:
https://blog.sonatype.com/central-repository-moving-to-https

Without this fix all PRs are blocked from testing for 4.11+ branches.

@yadvr
Copy link
Copy Markdown
Member Author

yadvr commented Jan 28, 2020

This will be forward merged to master/4.14 and then re-kick all Travis builds for PRs which are failing.

Travis has been failing lately as it requires that repositories use
https:// URLs.

Reference:
https://blog.sonatype.com/central-repository-moving-to-https

Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
@yadvr yadvr force-pushed the travis-maven-https-fix branch from 700a5fa to 0dbfb6e Compare January 28, 2020 07:08
@yadvr yadvr changed the base branch from 4.11 to 4.13 January 28, 2020 07:08
@yadvr yadvr closed this Jan 28, 2020
@yadvr yadvr reopened this Jan 28, 2020
Copy link
Copy Markdown
Contributor

@DaanHoogland DaanHoogland left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mvn says so so let's comply

Copy link
Copy Markdown
Member

@weizhouapache weizhouapache left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ohohoh, that's why many travis tests failed

Copy link
Copy Markdown
Member

@sateesh-chodapuneedi sateesh-chodapuneedi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@yadvr
Copy link
Copy Markdown
Member Author

yadvr commented Jan 28, 2020

I'll merge this, there seems to be an issue with ipmisim that's separate from this issue. (will check that ipmisim issue tomorrow, but merge this so all PRs are unblocked).

@yadvr yadvr merged commit b1783fb into apache:4.13 Jan 28, 2020
@yadvr yadvr added this to the 4.13.1.0 milestone Jan 28, 2020
@yadvr
Copy link
Copy Markdown
Member Author

yadvr commented Jan 29, 2020

cc @DaanHoogland @andrijapanicsb
Post merging, the ipmisim issue happened to due change in the dependency library pyghmi. To fix that I did a new release of ipmisim with a fixed pyghmi version: https://github.com/rhtyd/ipmisim
And uploaded the new ipmisim artifact here: https://pypi.org/project/ipmisim/0.8/

On Travis the oobm tests are failing, but likely due to Travis caching issue. I've tried to delete Travis caches, hopefully it will pass in the next run(s). Locally the tests are passing now the new ipmisim lib that gets installed via Marvin library against master/4.14:

=== TestName: test_oobm_background_powerstate_sync | Status : SUCCESS ===
ok
Tests out-of-band management configuration with valid data ... === TestName: test_oobm_configure_default_driver | Status : SUCCESS ===
ok
Tests out-of-band management configuration with invalid driver ... === TestName: test_oobm_configure_invalid_driver | Status : SUCCESS ===
ok
Tests out-of-band management host disable feature with ... === TestName: test_oobm_disable_feature_invalid | Status : SUCCESS ===
ok
Tests out-of-band management host disable feature with ... === TestName: test_oobm_disable_feature_valid | Status : SUCCESS ===
ok
Tests out-of-band management host enable feature with ... === TestName: test_oobm_enable_feature_invalid | Status : SUCCESS ===
ok
Tests out-of-band management host enable feature with ... === TestName: test_oobm_enable_feature_valid | Status : SUCCESS ===
ok
Tests out-of-band management enable/disable feature at cluster ... === TestName: test_oobm_enabledisable_across_clusterzones | Status : SUCCESS ===
ok
Tests out-of-band management issue power cycle action ... === TestName: test_oobm_issue_power_cycle | Status : SUCCESS ===
ok
Tests out-of-band management issue power off action ... === TestName: test_oobm_issue_power_off | Status : SUCCESS ===
ok
Tests out-of-band management issue power on action ... === TestName: test_oobm_issue_power_on | Status : SUCCESS ===
ok
Tests out-of-band management issue power reset action ... === TestName: test_oobm_issue_power_reset | Status : SUCCESS ===
ok
Tests out-of-band management issue power soft action ... === TestName: test_oobm_issue_power_soft | Status : SUCCESS ===
ok
Tests out-of-band management issue power action ... === TestName: test_oobm_issue_power_status | Status : SUCCESS ===
ok
Tests out-of-band management ownership expiry across multi-mgmt server ... === TestName: test_oobm_multiple_mgmt_server_ownership | Status : SUCCESS ===
ok
Tests out-of-band management change password feature ... === TestName: test_oobm_zchange_password | Status : SUCCESS ===
ok

----------------------------------------------------------------------
Ran 16 tests in 66.747s

OK

@yadvr
Copy link
Copy Markdown
Member Author

yadvr commented Jan 29, 2020

Confirmed fix, job3 passed for the rekicked Travis job:
https://travis-ci.org/apache/cloudstack/jobs/643301749

ustcweizhou pushed a commit to ustcweizhou/cloudstack that referenced this pull request Feb 28, 2020
Travis has been failing lately as it requires that repositories use
https:// URLs.

Reference:
https://blog.sonatype.com/central-repository-moving-to-https

Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants