Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

Fix CI builds failing due to invalid GPG keys. #19377

Merged
merged 3 commits into from
Oct 20, 2020

Conversation

josephevans
Copy link
Contributor

Description

We are having CI failures due to a GPG key expiring, which results in packages not being installed properly and our docker build scripts failing. This PR updates the key and repository to the latest.

[2020-10-19T18:51:21.002Z] Reading package lists...
[2020-10-19T18:51:21.002Z] W: GPG error: http://cran.rstudio.com/bin/linux/ubuntu trusty/ Release: The following signatures were invalid: KEYEXPIRED 1602869253 KEYEXPIRED 1602869253 KEYEXPIRED 1602869253
[2020-10-19T18:51:21.002Z] W: The repository 'http://cran.rstudio.com/bin/linux/ubuntu trusty/ Release' is not signed.
[2020-10-19T18:51:21.002Z] W: GPG error: https://cran.rstudio.com/bin/linux/ubuntu xenial/ InRelease: The following signatures were invalid: KEYEXPIRED 1602869253 KEYEXPIRED 1602869253 KEYEXPIRED 1602869253
[2020-10-19T18:51:21.002Z] W: The repository 'https://cran.rstudio.com/bin/linux/ubuntu xenial/ InRelease' is not signed.
[2020-10-19T18:51:21.002Z] E: Failed to fetch store:/var/lib/apt/lists/partial/cran.rstudio.com_bin_linux_ubuntu_trusty_Packages.gz Hash Sum mismatch
[2020-10-19T18:51:21.002Z] E: Some index files failed to download. They have been ignored, or old ones used instead.
[2020-10-19T18:51:21.002Z] + wget -qO /usr/lib/llvm-6.0/share/clang/run-clang-tidy.py https://raw.githubusercontent.com/llvm-mirror/clang-tools-extra/7654135f0cbd155c285fd2a37d87e27e4fff3071/clang-tidy/tool/run-clang-tidy.py
[2020-10-19T18:51:21.002Z] /usr/lib/llvm-6.0/share/clang/run-clang-tidy.py: No such file or directory
[2020-10-19T18:51:21.258Z] The command '/bin/sh -c /work/ubuntu_clang.sh' returned a non-zero code: 1

Checklist

Essentials

  • PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
  • Changes are complete (i.e. I finished coding on this PR)
  • All changes have test coverage
  • Code is well-documented

Changes

  • Feature1, tests, (and when applicable, API doc)
  • Feature2, tests, (and when applicable, API doc)

Comments

  • If this change is a backward incompatible change, why must this change be made.
  • Interesting edge cases to note here

@mxnet-bot
Copy link

Hey @josephevans , Thanks for submitting the PR
All tests are already queued to run once. If tests fail, you can trigger one or more tests again with the following commands:

  • To trigger all jobs: @mxnet-bot run ci [all]
  • To trigger specific jobs: @mxnet-bot run ci [job1, job2]

CI supported jobs: [edge, windows-gpu, unix-cpu, clang, centos-cpu, website, unix-gpu, sanity, windows-cpu, miscellaneous, centos-gpu]


Note:
Only following 3 categories can trigger CI :PR Author, MXNet Committer, Jenkins Admin.
All CI tests must pass before the PR can be merged.

@lanking520 lanking520 added the pr-awaiting-testing PR is reviewed and waiting CI build and test label Oct 19, 2020
@josephevans
Copy link
Contributor Author

Relevant post: https://www.bentoh.my/post/gpg-key-problem/

@lanking520 lanking520 added pr-work-in-progress PR is still work in progress and removed pr-awaiting-testing PR is reviewed and waiting CI build and test labels Oct 19, 2020
@lanking520 lanking520 added pr-awaiting-testing PR is reviewed and waiting CI build and test and removed pr-work-in-progress PR is still work in progress labels Oct 19, 2020
@lanking520 lanking520 added pr-work-in-progress PR is still work in progress pr-awaiting-testing PR is reviewed and waiting CI build and test and removed pr-awaiting-testing PR is reviewed and waiting CI build and test pr-work-in-progress PR is still work in progress labels Oct 19, 2020
@lanking520 lanking520 added pr-awaiting-testing PR is reviewed and waiting CI build and test pr-work-in-progress PR is still work in progress and removed pr-work-in-progress PR is still work in progress pr-awaiting-testing PR is reviewed and waiting CI build and test labels Oct 19, 2020
@lanking520 lanking520 added pr-work-in-progress PR is still work in progress pr-awaiting-testing PR is reviewed and waiting CI build and test and removed pr-awaiting-testing PR is reviewed and waiting CI build and test pr-work-in-progress PR is still work in progress labels Oct 19, 2020
Copy link
Contributor

@ChaiBapchya ChaiBapchya left a comment

Choose a reason for hiding this comment

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

Thanks a lot Joe!

@lanking520 lanking520 added pr-work-in-progress PR is still work in progress pr-awaiting-testing PR is reviewed and waiting CI build and test and removed pr-awaiting-testing PR is reviewed and waiting CI build and test pr-work-in-progress PR is still work in progress labels Oct 20, 2020
@lanking520 lanking520 added pr-awaiting-review PR is waiting for code review and removed pr-awaiting-testing PR is reviewed and waiting CI build and test labels Oct 20, 2020
@sandeep-krishnamurthy sandeep-krishnamurthy merged commit 653b480 into apache:v1.x Oct 20, 2020
@josephevans josephevans deleted the ci_fixes_v1.x branch October 20, 2020 17:10
josephevans added a commit to josephevans/mxnet that referenced this pull request Oct 20, 2020
* Update repo and key used for installing r. Should fix sanity builds in v1.x.

* Use correct repo (trusty) as v1.x branch uses ubuntu:16.04 for container. Also remove copy GPG key command.

* Remove all references to old expired key.

Co-authored-by: Joe Evans <[email protected]>
josephevans added a commit to josephevans/mxnet that referenced this pull request Oct 20, 2020
* Update repo and key used for installing r. Should fix sanity builds in v1.x.

* Use correct repo (trusty) as v1.x branch uses ubuntu:16.04 for container. Also remove copy GPG key command.

* Remove all references to old expired key.

Co-authored-by: Joe Evans <[email protected]>
samskalicky pushed a commit that referenced this pull request Oct 20, 2020
* Update repo and key used for installing r. Should fix sanity builds in v1.x.

* Use correct repo (trusty) as v1.x branch uses ubuntu:16.04 for container. Also remove copy GPG key command.

* Remove all references to old expired key.

Co-authored-by: Joe Evans <[email protected]>

Co-authored-by: Joe Evans <[email protected]>
samskalicky pushed a commit that referenced this pull request Oct 21, 2020
* Fix CI builds failing due to invalid GPG keys. (#19377)

* Update repo and key used for installing r. Should fix sanity builds in v1.x.

* Use correct repo (trusty) as v1.x branch uses ubuntu:16.04 for container. Also remove copy GPG key command.

* Remove all references to old expired key.

Co-authored-by: Joe Evans <[email protected]>

* Install cuda-toolkit-10-0 as required.

* Install the required packages and dependencies from path to avoid conflicting packages.

Co-authored-by: Joe Evans <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
pr-awaiting-review PR is waiting for code review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants