dentproject / dentOS

dentOS SwitchDev based NOS
Other
206 stars 59 forks source link

Quick workaround for dst root ca x3 expiration (kernel.org) #141

Closed sonoble closed 3 years ago

sonoble commented 3 years ago

Signed-off-by: Steve Noble snoble@sonn.com

Due to the expiration of the dst root certificate, dentOS will not build. The simplest work around is to update libgnutls30 to be able to contact kernel.org or any other site using the certificate.

https://letsencrypt.org/docs/dst-root-ca-x3-expiration-september-2021/

taraschornyiplv commented 3 years ago

@sonoble this is ok for now. However I think this should be part of updated builder image

sonoble commented 3 years ago

recheck

sonoble commented 3 years ago

@sonoble this is ok for now. However I think this should be part of updated builder image

I agree but we don't have the rights to push to the builder image currently, we may need to pivot to hosting the image in the linux foundation docker hub.

sonoble commented 3 years ago

The failure is weird, it says it is doing the 141 pull request but the changes say dbus.

PR #141 : Quick workaround for dst ro...

  Changes Add dbus package to the build (commit: aca3d27) (details / githubweb)
taraschornyiplv commented 3 years ago

this error is expected. for some reason merge job is started before PR is merged thus it builds what's in the main branch and not changes in PR,

sonoble commented 3 years ago

this error is expected. for some reason merge job is started before PR is merged thus it builds what's in the main branch and not changes in PR,

I see, it is doing two things, building main as is (which fails due to the certificate), and running builds on the PR separately. There are two verify builds running now (due to my request to recheck).

jwagantall commented 3 years ago

recheck

jwagantall commented 3 years ago

@sonoble , the triggers are fixed now.. the merge jobs should have not triggered in this case.

jwagantall commented 3 years ago

"DentOS merge" will still appear in the checks unfortunately since it was triggered before but is a non blocker for merging this PR