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

ipatests/azure: display actual dnf repo URLs #3366

Closed
wants to merge 1 commit into from
Closed

ipatests/azure: display actual dnf repo URLs #3366

wants to merge 1 commit into from

Conversation

fcami
Copy link
Contributor

@fcami fcami commented Jul 3, 2019

Display which dnf repositories were available at the prepare-build step via metalink.
Also display the fastestmirror cache.

Signed-off-by: François Cami fcami@redhat.com

@fcami fcami added the WIP Work in progress - not ready yet for review label Jul 3, 2019
Display which dnf repositories were available at the
prepare-build step via metalink.
Also display the fastestmirror cache.

Signed-off-by: François Cami <fcami@redhat.com>
@fcami fcami changed the title WIP ipatests/azure: display actual repo configuration WIP ipatests/azure: display actual dnf repo URLs Jul 3, 2019
@fcami fcami added the ipa-4-8 Mark for backport to ipa 4.8 label Jul 3, 2019
@fcami
Copy link
Contributor Author

fcami commented Jul 3, 2019

This change does not make builds slower as the makecache step is now much faster due to the added repolist step.

prepare-build took roughly 3m30s before this:
https://dev.azure.com/freeipa/freeipa/_build/results?buildId=511&view=logs&jobId=ca395085-040a-526b-2ce8-bdc85f692774
after this 3m28:
https://dev.azure.com/freeipa/freeipa/_build/results?buildId=519&view=logs&jobId=ca395085-040a-526b-2ce8-bdc85f692774

Build logs show that outbound rsync is blocked in azure and this is why we had failures.
Fastestmirror as introduced by 7027f79 should alleviate the issue once and for all.

@fcami fcami added needs review Pull Request is waiting for a review and removed WIP Work in progress - not ready yet for review labels Jul 4, 2019
@fcami fcami requested a review from netoarmando July 4, 2019 08:00
@fcami fcami changed the title WIP ipatests/azure: display actual dnf repo URLs ipatests/azure: display actual dnf repo URLs Jul 4, 2019
@flo-renaud flo-renaud self-assigned this Jul 8, 2019
@flo-renaud
Copy link
Contributor

Since the change does not impact the build time and it could help us troubleshoot in case of future issues, ACK.

@flo-renaud flo-renaud added ack Pull Request approved, can be merged and removed needs review Pull Request is waiting for a review labels Jul 8, 2019
@fcami fcami added the pushed Pull Request has already been pushed label Jul 8, 2019
@fcami
Copy link
Contributor Author

fcami commented Jul 8, 2019

master:

  • be7f54d ipatests/azure: display actual dnf repo URLs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged ipa-4-8 Mark for backport to ipa 4.8 pushed Pull Request has already been pushed
Projects
None yet
2 participants