Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update VM Images + Drop prior-ubuntu references #1444

Merged
merged 1 commit into from
Sep 16, 2021

Conversation

cevich
Copy link
Member

@cevich cevich commented Sep 14, 2021

These images contain a workaround for:
containers/podman#11123

Prior-Ubuntu support is being dropped everywhere.

Ref: containers/podman#11070
containers/automation_images#88

Signed-off-by: Chris Evich cevich@redhat.com

These images contain a workaround for:
     containers/podman#11123

Prior-Ubuntu support is being dropped everywhere.

Ref: containers/podman#11070
     containers/automation_images#88

Signed-off-by: Chris Evich <cevich@redhat.com>
cevich added a commit to cevich/image that referenced this pull request Sep 14, 2021
Mirror of changes from containers/skopeo#1444

These images contain a workaround for:
     containers/podman#11123

Prior-Ubuntu support is being dropped everywhere.

Ref: containers/podman#11070
     containers/automation_images#88

Signed-off-by: Chris Evich <cevich@redhat.com>
@cevich
Copy link
Member Author

cevich commented Sep 14, 2021

@mtrmac any idea what this means and if/how I can fix it:

---> Making bundle: validate-vet
go build github.com/containers/storage/pkg/devicemapper: invalid flag in pkg-config --cflags: /usr/bin/sh:
make: *** [Makefile:231: validate-local] Error 1

@mtrmac
Copy link
Contributor

mtrmac commented Sep 15, 2021

go build github.com/containers/storage/pkg/devicemapper: invalid flag in pkg-config --cflags: /usr/bin/sh:

Two guesses:

  • pkg-config missing (I see no reason to assume that)
  • The /bin/sh: warning: setlocale: LC_ALL: cannot change locale (en_US): No such file or directory things that are all over the log leaking into pkg-config output. (I see no reason to assume that either, but it seems more likely to happen unintentionally, e.g. by fedora:latest updating to a new release.)

Maybe see what pkg-config --cflags devmapper outputs in that environment.


BTW the “cannot change locale” warnings worry me somewhat; the system can probably mostly work in such an environment, but it just screams “untested error handling paths” to me. Would it be possible to fix that? I don’t very much care whether it is by changing the locale choice or by adding the locale data, though it might turn out to matter in practice.

@cevich
Copy link
Member Author

cevich commented Sep 15, 2021

Maybe see what pkg-config --cflags devmapper outputs in that environment.

Thanks for the hint, that is something I can check...

but it just screams “untested error handling paths” to me. Would it be possible to fix that?

Yeah I noticed that too, and it seems new. I think the first step to a fix is understanding the cause...for which I have less than zero insight. Do you have any idea where that error is coming from? I cannot find references to setlocate anywhere under hack/.

@cevich
Copy link
Member Author

cevich commented Sep 15, 2021

...Oh right, so make validate runs inside the container. That's where (and maybe why?) we see the locale errors. This is totally reproducible locally, it doesn't require CI:

$ make validate
podman run --rm -i -e TESTFLAGS="-v -check.v -test.timeout=15m" -e CI= -e SKOPEO_CONTAINER_TESTS=1 -t --security-opt label=disable -v /home/cevich/devel/skopeo:/src/github.com/containers/skopeo -w /src/github.com/containers/skopeo quay.io/libpod/skopeo_cidev:c6431352024203264 make validate-local
Trying to pull quay.io/libpod/skopeo_cidev:c6431352024203264...
Getting image source signatures
Copying blob ecfb9899f4ce done
...cut...
/bin/sh: warning: setlocale: LC_ALL: cannot change locale (en_US): No such file or directory
...cut...
$ podman run -it --rm quay.io/libpod/skopeo_cidev:c6431352024203264
bash: warning: setlocale: LC_ALL: cannot change locale (en_US): No such file or directory
/usr/bin/sh: warning: setlocale: LC_ALL: cannot change locale (en_US): No such file or directory
/usr/bin/sh: warning: setlocale: LC_ALL: cannot change locale (en_US): No such file or directory
/usr/bin/sh: warning: setlocale: LC_ALL: cannot change locale (en_US): No such file or directory
[root@76cb0cc7dcaf /]# cat /etc/os-release
NAME=Fedora
VERSION="34 (Container Image)"
ID=fedora
VERSION_ID=34
VERSION_CODENAME=""
PLATFORM_ID="platform:f34"
...
[root@76cb0cc7dcaf /]# pkg-config --cflags devmapper

@cevich
Copy link
Member Author

cevich commented Sep 15, 2021

Ahh ha! I found it. I set this up in the Containerfile to make perl happy, apparently that broke everything else. Working on a manual fix.

@cevich
Copy link
Member Author

cevich commented Sep 15, 2021

...okay, I fixed this in a really hacky/not-recommended way. I built a new image with a fix, then manually overwrote the tag and shoved it up into quay (after confirming the localle errors are gone). Re-running the tests now.

@rhatdan
Copy link
Member

rhatdan commented Sep 16, 2021

LGTM

@rhatdan rhatdan merged commit 31b8981 into containers:main Sep 16, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 22, 2023
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants