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

builds are broken: go-md2man unavailable #4632

Closed
edsantiago opened this issue Dec 3, 2019 · 3 comments · Fixed by #4635
Closed

builds are broken: go-md2man unavailable #4632

edsantiago opened this issue Dec 3, 2019 · 3 comments · Fixed by #4635
Assignees
Labels
kind/bug Categorizes issue or PR as related to a bug. locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. ok-to-test

Comments

@edsantiago
Copy link
Member

#4616 broke rawhide -- and, presumably, all koji/brew -- builds:

if [ ! -x "/builddir/build/BUILD/libpod-5c3af009c6027429d60f6e73bf3ade346ec3d410/_output/bin/go-md2man" ]; then \
	   	env GO111MODULE=off go get -u github.com/cpuguy83/go-md2man; \
fi
# cd .; git clone -- https://github.com/cpuguy83/go-md2man /builddir/build/BUILD/libpod-5c3af009c6027429d60f6e73bf3ade346ec3d410/_output/src/github.com/cpuguy83/go-md2man
Cloning into '/builddir/build/BUILD/libpod-5c3af009c6027429d60f6e73bf3ade346ec3d410/_output/src/github.com/cpuguy83/go-md2man'...
fatal: unable to access 'https://github.com/cpuguy83/go-md2man/': Could not resolve host: github.com
package github.com/cpuguy83/go-md2man: exit status 128
make: *** [Makefile:490: .install.md2man] Error 1
error: Bad exit status from /var/tmp/rpm-tmp.953d7H (%install)

Sorry, but it will have to be reverted. The koji/brew build environment is network-isolated and cannot run go fetch.

@edsantiago edsantiago added kind/bug Categorizes issue or PR as related to a bug. priority/critical-urgent labels Dec 3, 2019
@TomSweeneyRedHat
Copy link
Member

UGH, my main concern here is how the PR to make this change passed. We must not use changes in Makefile that are in a PR when testing the PR?

@edsantiago
Copy link
Member Author

The PR passed because the build happens in CI-land, not koji-land. Koji-land is different. This really couldn't have been caught in CI.

@ssbarnea ssbarnea self-assigned this Dec 3, 2019
@ssbarnea
Copy link
Collaborator

ssbarnea commented Dec 3, 2019

More than happy to help address this. Still, the reality is that we are talking about a regression happening on a specific case that is not covered in CI. That is the real bug: if we want to avoid this from happening we need to be sure we have a CI job running that runs in an isolated environment. That would be a good start.

Apparently my PR uncovered a but at https://github.com/containers/libpod/blob/master/Makefile#L490 where the desired behavior should be install command if not found anywhere -- as it could be installed at system level via a rpm.

I will make a PR to address this, is likely a one-liner.

@github-actions github-actions bot added the locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. label Sep 23, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
kind/bug Categorizes issue or PR as related to a bug. locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. ok-to-test
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants