Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cherry-pick #18005 to 7.x: Disable repository expiration checks in Journalbeat builds #18016

Merged
merged 1 commit into from
Apr 27, 2020

Conversation

jsoriano
Copy link
Member

@jsoriano jsoriano commented Apr 27, 2020

Cherry-pick of PR #18005 to 7.x branch. Original message:

Journalbeat builds are failing in all branches with errors like the following one:

W: GPG error: http://archive.debian.org jessie Release: The following signatures were invalid: KEYEXPIRED 1587841717
WARNING: The following packages cannot be authenticated!
  liblzma5:i386 libgpg-error0:i386
E: There are problems and -y was used without --force-yes
Error: running "apt-get install -y --no-install-recommends -o Acquire::Check-Valid-Until=false libsystemd-dev:i386 libsystemd0:i386 libgcrypt20:i386" failed with exit code 100

Jessie repositories are not updated anymore and their keys got outdated a couple of days ago.

# apt-key list | grep expired
pub   4096R/46925553 2012-04-27 [expired: 2020-04-25]
pub   4096R/65FFB764 2012-05-08 [expired: 2019-05-07]

I tried to disable only expiration checks with -o Acquire::Check-Valid-Until=false but it was not enough.

I am disabling all authentication checks for packages installation during Journalbeat builds. We should review this in a follow up, so we don't build with unauthenticated packages.

@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Apr 27, 2020
@jsoriano jsoriano requested a review from mikemadden42 April 27, 2020 16:29
@jsoriano jsoriano self-assigned this Apr 27, 2020
@mikemadden42
Copy link
Contributor

run beats-ci/package

Copy link
Contributor

@mikemadden42 mikemadden42 left a comment

Choose a reason for hiding this comment

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

LGTM

@mikemadden42 mikemadden42 merged commit 3d7f736 into elastic:7.x Apr 27, 2020
@andresrc andresrc removed the needs_team Indicates that the issue/PR needs a Team:* label label May 2, 2020
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