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

[ipa-4-7] Travis: Enable IPv6 support for Docker #4307

Closed
wants to merge 1 commit into from
Closed

[ipa-4-7] Travis: Enable IPv6 support for Docker #4307

wants to merge 1 commit into from

Conversation

netoarmando
Copy link
Member

Environment variable TRAVIS is not accessible inside the container anymore, passing it to the container didn't have the same effect expected by the hack this commit removes. However, enabling IPv6, as described in travis-ci/travis-ci#8891, was effective to execute the tests.

Issue: https://pagure.io/freeipa/issue/8213

Signed-off-by: Armando Neto abiagion@redhat.com

Environment variable 'TRAVIS' is not accessible inside the container
anymore, passing it to the container didn't have the same effect
expected by the hack this commit removes. However, enabling IPv6, as
described in travis-ci/travis-ci#8891,
was effective to execute the tests.

Issue: https://pagure.io/freeipa/issue/8213

Signed-off-by: Armando Neto <abiagion@redhat.com>
@flo-renaud flo-renaud self-assigned this Mar 5, 2020
@flo-renaud flo-renaud added the ack Pull Request approved, can be merged label Mar 5, 2020
@flo-renaud
Copy link
Contributor

Hi @netoarmando
thanks for the PR, LGTM as Travis is now working as expected

@flo-renaud flo-renaud added the pushed Pull Request has already been pushed label Mar 5, 2020
@flo-renaud
Copy link
Contributor

ipa-4-7:

  • 4cddbfd Travis: Enable IPv6 support for Docker

@flo-renaud flo-renaud closed this Mar 5, 2020
@netoarmando netoarmando deleted the ipa-4-7-travis-enable-ipv6 branch March 5, 2020 12:01
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 pushed Pull Request has already been pushed
Projects
None yet
2 participants