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

docker_image_rpm: Do not add devel repos in TW snapshot tests #6047

Merged
merged 1 commit into from
Oct 30, 2018

Conversation

Vogtinator
Copy link
Member

That makes no sense if the objective is to test whether the image works.

Untested.

That makes no sense if the objective is to test whether the image works.
@okurz
Copy link
Member

okurz commented Oct 26, 2018

@pdostal

@pdostal
Copy link
Member

pdostal commented Oct 29, 2018

Hello, the change looks good but I'm not really sure if that will pass. Please @Vogtinator can you provide a test run?

@Vogtinator
Copy link
Member Author

@pdostal Why shouldn't it pass?

@Vogtinator
Copy link
Member Author

Test run: http://10.160.67.86/tests/160#step/docker_image_rpm/14

I ran it in a livecd test as it's quicker and it succeeded until it ran out of space after the test.

@pdostal
Copy link
Member

pdostal commented Oct 29, 2018

OK, the out of space error seems to be related to the live CD environment and the package seems to be all right. Thank you @Vogtinator for your effort!

@ggardet
Copy link
Collaborator

ggardet commented Oct 30, 2018

@Vogtinator what about the current vs latest tags? It is currently the problem in openQA: https://openqa.opensuse.org/tests/784346#step/docker_image_rpm/22

@okurz okurz merged commit 9ce13a1 into os-autoinst:master Oct 30, 2018
@Vogtinator
Copy link
Member Author

@ggardet That's exactly what this PR fixes. By using the image from the wrong location, the tags are obviously wrong as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants