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

Backport OpenShift OKD docs for 1.8 #13443

Merged
merged 4 commits into from
Oct 8, 2020
Merged

Conversation

errordeveloper
Copy link
Contributor

@errordeveloper errordeveloper commented Oct 7, 2020

Backport #13439 & #13439 to 1.8. Depends on #13448.

@errordeveloper errordeveloper requested a review from a team as a code owner October 7, 2020 12:17
@maintainer-s-little-helper maintainer-s-little-helper bot added backport/1.8 kind/backports This PR provides functionality previously merged into master. labels Oct 7, 2020
@errordeveloper errordeveloper marked this pull request as draft October 7, 2020 12:17
errordeveloper and others added 3 commits October 8, 2020 10:27
[ upstream commit 43e6a4a ]

Co-authored-by: Vadim Rutkovsky <roignac@gmail.com>
Co-authored-by: Joe Stringer <joe@cilium.io>
Signed-off-by: Ilya Dmitrichenko <errordeveloper@gmail.com>
[ upstream commit 2155815 ]

Signed-off-by: Ilya Dmitrichenko <errordeveloper@gmail.com>
Signed-off-by: Ilya Dmitrichenko <errordeveloper@gmail.com>
@errordeveloper errordeveloper force-pushed the pr/v1.8-backport-2020-10-07-okd-docs branch from e429fff to 4175d67 Compare October 8, 2020 09:27
@errordeveloper errordeveloper marked this pull request as ready for review October 8, 2020 09:27
The rest of the guide relies on the use of `cilium-test` namespace
which was added to connectivity test after 1.8. This simply backports
that improvement inline just for the OpenShift OKD guide.

Signed-off-by: Ilya Dmitrichenko <errordeveloper@gmail.com>
@errordeveloper errordeveloper added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Oct 8, 2020

.. note::

Version ``1.8-dev`` has bug fixes for OpenShift, those fixes will get included in 1.8.5.
Copy link
Member

@gandro gandro Oct 8, 2020

Choose a reason for hiding this comment

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

Shouldn't we just recommend here that users simply deploy least Cilium 1.8.5? I know it has not been released yet, but I find it a bit weird to refer to development version in the stable docs.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Due to popular demand the intention is to give a preview to folks ASAP, hence it's even being backported :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's a bit as if we made an RC for 1.8.5, but it turns out we don't do RCs for patch releases. This is per discussion with @joestringer and @aanm.

@gandro gandro merged commit 903eb43 into v1.8 Oct 8, 2020
@gandro gandro deleted the pr/v1.8-backport-2020-10-07-okd-docs branch October 8, 2020 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/backports This PR provides functionality previously merged into master. ready-to-merge This PR has passed all tests and received consensus from code owners to merge.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants