Skip to content
This repository has been archived by the owner on Feb 24, 2020. It is now read-only.

release 1.30.0 #3921

Merged
merged 3 commits into from
Apr 16, 2018
Merged

release 1.30.0 #3921

merged 3 commits into from
Apr 16, 2018

Conversation

iaguis
Copy link
Member

@iaguis iaguis commented Apr 11, 2018

🐙

@iaguis iaguis added this to the v1.30.0 milestone Apr 11, 2018
@iaguis iaguis requested a review from lucab April 11, 2018 12:28
CHANGELOG.md Outdated

- status: added read from uuid-file ([#3860](https://github.com/rkt/rkt/pull/3860)).
- stage0/run: relax '--hosts-entry' parser ([#3833](https://github.com/rkt/rkt/pull/3833)).
- iottymux: Store logs for kubelet in the appropriate location ([#3798](https://github.com/rkt/rkt/pull/3798)). This change is made for rktlet. iottymux will store the logs directly in the CRI format.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lowercase store.

@iaguis iaguis force-pushed the iaguis/release-1.30.0 branch from 907ebcb to 0cd99c1 Compare April 11, 2018 13:17
@taqtiqa-mark taqtiqa-mark mentioned this pull request Apr 11, 2018
@lucab
Copy link
Member

lucab commented Apr 12, 2018

On a recent build system, stage1-kvm now fails to build with:

util/memfd.c:40:12: error: static declaration of ‘memfd_create’ follows non-static declaration
 static int memfd_create(const char *name, unsigned int flags)
            ^~~~~~~~~~~~
In file included from /usr/include/x86_64-linux-gnu/bits/mman-linux.h:115:0,
                 from /usr/include/x86_64-linux-gnu/bits/mman.h:45,
                 from /usr/include/x86_64-linux-gnu/sys/mman.h:41,
                 from /home/.../src/github.com/rkt/rkt/build-rkt-1.30.0/tmp/usr_from_kvm/qemu/src/include/sysemu/os-posix.h:29,
                 from /home/.../src/github.com/rkt/rkt/build-rkt-1.30.0/tmp/usr_from_kvm/qemu/src/include/qemu/osdep.h:104,
                 from util/memfd.c:28:
/usr/include/x86_64-linux-gnu/bits/mman-shared.h:46:5: note: previous declaration of ‘memfd_create’ was here
 int memfd_create (const char *__name, unsigned int __flags) __THROW;
     ^~~~~~~~~~~~
make[1]: *** [util/memfd.o] Error 1

@iaguis
Copy link
Member Author

iaguis commented Apr 13, 2018

Weird, I've built a release successfully following the release guide. I get this is not building inside rkt, right?

I tried building directly on my laptop and I get an earlier error when building qemu:

set -e;   cd /home/iaguis/work/go/src/github.com/rkt/rkt/build-rkt/tmp/usr_from_kvm/qemu/src; ./configure --disable-bsd-user --disable-docs --disable-guest-agent --disable-guest-agent-msi --disable-sdl --disable-gtk --disable-vte --disable-curses --disable-cocoa --disable-brlapi --disable-vnc --disable-seccomp --disable-curl --disable-bluez --disable-cap-ng --disable-rbd --disable-libiscsi --disable-libnfs --disable-smartcard --disable-libusb --disable-glusterfs --disable-archipelago --disable-tcmalloc --disable-jemalloc --disable-debug-info --static --enable-virtfs --target-list=x86_64-softmmu --python=/usr/bin/python2 --disable-werror ; touch "/home/iaguis/work/go/src/github.com/rkt/rkt/build-rkt/stamps/__stage1_usr_from_kvm_qemu_mk_conf.stamp"

ERROR: sizeof(size_t) doesn't match GLIB_SIZEOF_SIZE_T.
       You probably need to set PKG_CONFIG_LIBDIR
       to point to the right pkg-config files for your
       build target

make: *** [stage1/usr_from_kvm/qemu.mk:54: /home/iaguis/work/go/src/github.com/rkt/rkt/build-rkt/stamps/__stage1_usr_from_kvm_qemu_mk_conf.stamp] Error 1

It seems something is wrong with my glib headers' config but I haven't looked further into it.

@lucab
Copy link
Member

lucab commented Apr 13, 2018

Yes, this is outside of the rkt-builder. I was just doing the usual pre-flight check on my local machine before building the release artifact. I think I have a fix for that, looking at it now.

Your error is unrelated, I seem to remember having seen that already. I remember that it was related to arch-linux not shipping one of the static libraries qemu autoconf is looking for, but I don't remember which.

lucab added a commit to lucab/rkt that referenced this pull request Apr 13, 2018
This is a backport of a qemu upstream patch to fix a memfd related
failure. It first adds infrastructure to apply the patch to qemu at
build time.

Ref: https://git.qemu.org/?p=qemu.git;a=commit;h=75e5b70e6b5dcc4f2219992d7cffa462aa406af0
Ref: rkt#3921 (comment)
@lucab
Copy link
Member

lucab commented Apr 13, 2018

@iaguis #3922 fixes that build failure.

In the meanwhile, docker2aci regressed with go-1.10 at appc/docker2aci#259. Thus I'd like to bump docker2aci in here with the fix before tagging v1.30.0.

@iaguis
Copy link
Member Author

iaguis commented Apr 13, 2018

In the meanwhile, docker2aci regressed with go-1.10 at appc/docker2aci#259. Thus I'd like to bump docker2aci in here with the fix before tagging v1.30.0.

OK. Interesting issue btw :)

lucab added a commit to lucab/rkt that referenced this pull request Apr 13, 2018
This is a backport of a qemu upstream patch to fix a memfd related
failure. It first adds infrastructure to apply the patch to qemu at
build time.

Ref: https://git.qemu.org/?p=qemu.git;a=commit;h=75e5b70e6b5dcc4f2219992d7cffa462aa406af0
Ref: rkt#3921 (comment)
@iaguis iaguis force-pushed the iaguis/release-1.30.0 branch from 0cd99c1 to 757593e Compare April 13, 2018 17:56
@iaguis iaguis force-pushed the iaguis/release-1.30.0 branch from 757593e to 6596ced Compare April 13, 2018 17:57
@iaguis
Copy link
Member Author

iaguis commented Apr 13, 2018

Rebased to include #3922 and #3923

@lucab lucab modified the milestones: v1.30.0, v1+ Apr 13, 2018
@taqtiqa-mark
Copy link

@lucab, regarding your comment about brining in docker2aci:
Would theses changes resolve the Buildah compatibility issue inappc/docker2aci/issues/257. Like the users in that issue I'm using woofwoofinc's build of docker2aci.

Related, and in case it confirms anything you've seen: with rkt fetch... then rkt image export... I get images that are about twice the size of what I get from skopeo copy... then (Woof Woof Inc) docker2aci ....
I'm still trying to get time to make a minimal rep example, but thought to mention it in case it correlates with anything you've seen/experienced.

@lucab
Copy link
Member

lucab commented Apr 15, 2018

@taqtiqa-mark no, those are unrelated and only fix a go-1.10 regression, see #3923.

Copy link
Member

@lucab lucab left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lucab lucab merged commit 601d988 into rkt:master Apr 16, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants