Skip to content

Conversation

@fedepaol
Copy link
Member

The images pulled from quay lack of multi arch support, making the frr mode unusable on ARM (for RPis, for example). Here we revert back to dockerhub.

@fedepaol
Copy link
Member Author

Fixes #1216

@liornoy
Copy link
Contributor

liornoy commented Mar 23, 2022

There is another one referencing FRR from quay.io on e2etest/pkg/frr/container/container.go

@sabinaaledort
Copy link
Contributor

what about the dockerhub rate limit? did something change there?

@fedepaol
Copy link
Member Author

what about the dockerhub rate limit? did something change there?

Afaik, the dockerhub rate limit was something affecting only the downstream CI before switching back to the custom builds.
The images on quay are not the frr official ones (see FRRouting/frr#10587), so I think it makes sense to revert to dockerhub.

@sabinaaledort
Copy link
Contributor

/lgtm

The images pulled from quay lack of multi arch support, making the frr
mode unusable on ARM (for RPis, for example). Here we revert back to
dockerhub.
@fedepaol
Copy link
Member Author

There is another one referencing FRR from quay.io on e2etest/pkg/frr/container/container.go

Fixed, thanks

@liornoy
Copy link
Contributor

liornoy commented Mar 23, 2022

lgtm

@fedepaol
Copy link
Member Author

Closing this due to the drift of main.
Will file one for 0.12 and another one for main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants