[opensuse-factory] NGINX Ingress Controller for Kubernetes - new packages
Hi, I recently submitted 17 packages to Factory and I would like to explain the purpose of those requests. Kubernetes has a concept of Ingress[0] which allows to expose a service running inside cluster to the external network. Although to be able to use that functionality, user needs to deploy the Ingress Controller[1]. The most popular, open source and upstream supported implementation is NGINX Ingress Controller[2]. I would like to bring the support of Ingress in Kubic and SUSE CaaSP projects, which means that we need a container image and packages for every component needed by NGINX Ingress Controller. That's why I started with submitting all needed packages to Factory. This is the list of submit requests I made: https://build.opensuse.org/request/show/617519 https://build.opensuse.org/request/show/617660 https://build.opensuse.org/request/show/617664 https://build.opensuse.org/request/show/617665 https://build.opensuse.org/request/show/617667 https://build.opensuse.org/request/show/617668 https://build.opensuse.org/request/show/617669 https://build.opensuse.org/request/show/617670 https://build.opensuse.org/request/show/617671 https://build.opensuse.org/request/show/617672 https://build.opensuse.org/request/show/617673 https://build.opensuse.org/request/show/617731 https://build.opensuse.org/request/show/617734 https://build.opensuse.org/request/show/617745 https://build.opensuse.org/request/show/617750 https://build.opensuse.org/request/show/617752 https://build.opensuse.org/request/show/617753 I would be very thankful for feedback and reviews. Cheers, Michal [0] https://kubernetes.io/docs/concepts/services-networking/ingress/ [1] https://kubernetes.io/docs/concepts/services-networking/ingress/#ingress-con... [2] https://github.com/kubernetes/ingress-nginx -- To unsubscribe, e-mail: opensuse-factory+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse-factory+owner@opensuse.org
On 06/19/2018 03:45 PM, Michal Rostecki wrote:
Hi,
I recently submitted 17 packages to Factory and I would like to explain the purpose of those requests.
Kubernetes has a concept of Ingress[0] which allows to expose a service running inside cluster to the external network. Although to be able to use that functionality, user needs to deploy the Ingress Controller[1]. The most popular, open source and upstream supported implementation is NGINX Ingress Controller[2].
I would like to bring the support of Ingress in Kubic and SUSE CaaSP projects, which means that we need a container image and packages for every component needed by NGINX Ingress Controller. That's why I started with submitting all needed packages to Factory.
This is the list of submit requests I made:
https://build.opensuse.org/request/show/617519 https://build.opensuse.org/request/show/617660 https://build.opensuse.org/request/show/617664 https://build.opensuse.org/request/show/617665 https://build.opensuse.org/request/show/617667 https://build.opensuse.org/request/show/617668 https://build.opensuse.org/request/show/617669 https://build.opensuse.org/request/show/617670 https://build.opensuse.org/request/show/617671 https://build.opensuse.org/request/show/617672 https://build.opensuse.org/request/show/617673 https://build.opensuse.org/request/show/617731 https://build.opensuse.org/request/show/617734 https://build.opensuse.org/request/show/617745 https://build.opensuse.org/request/show/617750 https://build.opensuse.org/request/show/617752 https://build.opensuse.org/request/show/617753
I would be very thankful for feedback and reviews.
Cheers, Michal
[0] https://kubernetes.io/docs/concepts/services-networking/ingress/ [1] https://kubernetes.io/docs/concepts/services-networking/ingress/#ingress-con... [2] https://github.com/kubernetes/ingress-nginx
Hi Michal. I added some comments. Mostly about using the license macro to install the LICENSE. This is something we are doing in order to make sure we are installing he license file in the container images, thus it would be great if you could fix it. A part from that, I added a couple of comments more that you can see in the requests. Thanks jordi -- To unsubscribe, e-mail: opensuse-factory+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse-factory+owner@opensuse.org
On Tue, 2018-06-19 at 15:45 +0200, Michal Rostecki wrote:
Hi,
I would be very thankful for feedback and reviews. Hi
As commented in sr [1]: * In case package like nginx-module-devel-kit-source has to exist, it would be nice to have it explained in form of i.e comment * It is strongly recommended to use macro for %configure, it is not used: +cd nginx +./configure \ If it is not suitable, comment explaining this should be present * "nginx-module-devel-kit.rpmlintrc" should be "nginx-module-devel-kit- rpmlintrc" - I wonder why automatic checks did not catch this Cheers Martin 1. https://build.opensuse.org/request/show/617519
Thanks for everyone who commented and reviewed my requests! Currently, the following requests are still in review. I would be very thankful for the feedback or acceptance. https://build.opensuse.org/request/show/620238 https://build.opensuse.org/request/show/620516 https://build.opensuse.org/request/show/620514 https://build.opensuse.org/request/show/620510 https://build.opensuse.org/request/show/620508 https://build.opensuse.org/request/show/620506 https://build.opensuse.org/request/show/620501 https://build.opensuse.org/request/show/620498 https://build.opensuse.org/request/show/620495 On 06/21/2018 03:06 PM, Martin Pluskal wrote:
On Tue, 2018-06-19 at 15:45 +0200, Michal Rostecki wrote:
Hi,
I would be very thankful for feedback and reviews. Hi
As commented in sr [1]: * In case package like nginx-module-devel-kit-source has to exist, it would be nice to have it explained in form of i.e comment
Comments added in all requests which are still in review. I also added links to this ML thread.
* It is strongly recommended to use macro for %configure, it is not used: +cd nginx +./configure \ If it is not suitable, comment explaining this should be present * "nginx-module-devel-kit.rpmlintrc" should be "nginx-module-devel-kit- rpmlintrc" - I wonder why automatic checks did not catch this
Using %configure macro for nginx modules is impossible, because nginx's configure script doesn't support some options that this macro is using (i.e. --host). That's why the main nginx package doesn't use %configure macro as well. Cheers, Michal
participants (3)
-
Jordi Massaguer Pla -
Martin Pluskal -
Michal Rostecki