Conversation
|
@ldemailly: Adding do-not-merge/release-note-label-needed because the release note process has not been followed. DetailsOne of the following labels is required "release-note", "release-note-action-required", or "release-note-none". Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: rkpagadala Assign the PR to them by writing The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these OWNERS Files:You can indicate your approval by writing |
|
there is debate on holding off on changing this pending kubecon demos etc |
|
I think postponing this a week, especially given that everyone will be on travel, etc., makes sense, even if there is a workaround. Should we target Dec 6 (after istio summit / workshop) or Dec 9 (after kubecon)? |
|
Updating the revision was an oversight and normally done as part of a release. The eventual direction is probably to move towards cutting down on the # of files that need to get updated, possibly by grabbing the version # from another location or leveraging the result of https://api.github.com/repos/istio/istio/releases/latest |
|
Folks, please hold off this until it is well tested. I won't have time to do a personal spin on this till after kubecon. We have a lab coming up and many interest in istio next week, we would want everyone use the most stable release. |
linsun
left a comment
There was a problem hiding this comment.
see comment earlier, need more time.
|
I’ve been building istio training around 0.3.0 for next week. Things seem
pretty good so far!
On Fri, Dec 8, 2017 at 2:56 PM Laurent Demailly ***@***.***> wrote:
@linsun <https://github.com/linsun> can this (or #1977
<#1977>) go in now ? kubecon is over ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#1940 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AADP0dqGjqLxqC1eR2GAyxq_OwqnsXxQks5s-bCAgaJpZM4QxCsM>
.
--
*Christian Posta*
twitter: @christianposta
http://www.christianposta.com/blog
http://fabric8.io
|
linsun
left a comment
There was a problem hiding this comment.
I did a quick testing on IBM Cloud Container Service (k8s 1.7.4) it works at the first glance. Thank you for holding the move for kubecon, appreciate that!
ps: shouldn't this be part of finalize