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

Update deb/rpm manifests #442

Closed
luxas opened this issue Sep 14, 2017 · 3 comments
Closed

Update deb/rpm manifests #442

luxas opened this issue Sep 14, 2017 · 3 comments
Assignees
Labels
area/releasing priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now.
Milestone

Comments

@luxas
Copy link
Member

luxas commented Sep 14, 2017

Basically flipping the versions to v1.8.0 when released and updating the kubelet manifests to match https://github.com/kubernetes/kubernetes/blob/master/build/debs/kubeadm-10.conf

We might also have to run systemctl daemon-reload in kubeadm or kubelet postinst as discussed with @jessicaochen

@pipejakob can you please take ownership over getting this done and pushing the debs on release day?

@luxas luxas added area/releasing priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. labels Sep 14, 2017
@luxas luxas added this to the v1.8 milestone Sep 14, 2017
@pipejakob
Copy link
Contributor

I accept this grave responsibility. :-)

@pipejakob
Copy link
Contributor

I'm going to work on adding this to the deb/rpm release tooling so that we don't accidentally bundle the wrong version of kubeadm-10.conf with different releases (since we use the same scripts to still release 1.6.x and 1.7.x patches).

CC @mikedanese

pipejakob added a commit to pipejakob/release that referenced this issue Sep 26, 2017
For kubeadm 1.8, we changed the flags we want to start kubelet with.
This change breaks 10-kubeadm.conf into two files (pre- and post-1.8)
and selects which one to use at build time, based on the target version.

I've also included unit tests for the new functionality, but they aren't
run by default ("go test" manually), but are meant to demonstrate the
versioning edge cases I tested.

A similar change still needs to happen for our rpm build scripts.

kubernetes/kubeadm#442
pipejakob added a commit to pipejakob/release that referenced this issue Sep 26, 2017
For kubeadm 1.8, we changed the flags we want to start kubelet with.
This change breaks 10-kubeadm.conf into two files (pre- and post-1.8)
and selects which one to use at build time, based on the target version.

I've also included unit tests for the new functionality, but they aren't
run by default ("go test" manually), but are meant to demonstrate the
versioning edge cases I tested.

A similar change still needs to happen for our rpm build scripts.

kubernetes/kubeadm#442
pipejakob added a commit to pipejakob/release that referenced this issue Sep 26, 2017
For kubeadm 1.8, we changed the flags we want to start kubelet with.
This change breaks 10-kubeadm.conf into two files (pre- and post-1.8)
and selects which one to use at build time, based on the target version.

I've also included unit tests for the new functionality which aren't run
by default ("go test" manually), but are meant to demonstrate the
versioning edge cases I tested.

A similar change still needs to happen for our rpm build scripts.

kubernetes/kubeadm#442
pipejakob added a commit to pipejakob/release that referenced this issue Sep 27, 2017
For kubeadm 1.8, we changed the flags we want to start kubelet with.
This change breaks 10-kubeadm.conf into two files (pre- and post-1.8)
and selects which one to use at build time, based on the target version.

I've also included unit tests for the new functionality which aren't run
by default ("go test" manually), but are meant to demonstrate the
versioning edge cases I tested.

A similar change still needs to happen for our rpm build scripts.

kubernetes/kubeadm#442
@luxas
Copy link
Member Author

luxas commented Sep 29, 2017

@pipejakob addressed this some days ago in kubernetes/release#413
Closing...

@luxas luxas closed this as completed Sep 29, 2017
justaugustus pushed a commit to justaugustus/release that referenced this issue Jun 10, 2019
For kubeadm 1.8, we changed the flags we want to start kubelet with.
This change breaks 10-kubeadm.conf into two files (pre- and post-1.8)
and selects which one to use at build time, based on the target version.

I've also included unit tests for the new functionality which aren't run
by default ("go test" manually), but are meant to demonstrate the
versioning edge cases I tested.

A similar change still needs to happen for our rpm build scripts.

kubernetes/kubeadm#442
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/releasing priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now.
Projects
None yet
Development

No branches or pull requests

3 participants