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

Support multi-domain, ECC certificates, cluster mode #9

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

fr0der1c
Copy link

I implemented the following feature and this branch is working well on my cluster, serving more than 10 ingresses across namespaces:

  • Multi-domain support
  • ECC certificate support
  • Cluster mode
  • Use full chain certificate to have compatibility with command-line tools like cURL

@math-nao math-nao self-assigned this Sep 16, 2019
@vkruoso
Copy link

vkruoso commented Sep 19, 2019

Testing the previous version right now. Was going to open an issue to have full cluster support, but this is already implemented here. Any ideas on the timeframe to a release?

@math-nao
Copy link
Owner

It should be done in the following days.


if [ "${STATUS_CODE}" = "200" ]; then
# try get namespaces (cluster mode)
status_code=$(k8s_api_call "GET" /api/v1/namespaces 2>"${RES_FILE}")
Copy link
Owner

Choose a reason for hiding this comment

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

Deployment cluster scoped will failed. Please update your code to manage this case.
Use local keyword for variables when needed.

Copy link
Author

@fr0der1c fr0der1c Sep 25, 2019

Choose a reason for hiding this comment

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

Deployment cluster scoped will failed. Please update your code to manage this case.

What error did you see?

Copy link
Author

Choose a reason for hiding this comment

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

Use local keyword for variables when needed.

I don't think sh supports local.

Copy link
Owner

@math-nao math-nao Sep 25, 2019

Choose a reason for hiding this comment

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

What error did you see?

Cluster scoped deployment will failed due to a call requesting all namespaces without having a clusterrole. Just try it.

I don't think sh supports local.

ash is the system shell used on busybox and local keyword are supported.

Copy link
Author

@fr0der1c fr0der1c Sep 26, 2019

Choose a reason for hiding this comment

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

Cluster scoped deployment will failed due to a call requesting all namespaces without having a clusterrole. Just try it.

If you set cluster to true you will have a ClusterRole. If you set cluster to false(which is default), the request will fail. This is expected and will not cause problems. I've tested on my cluster. Maybe you need a fresh install? See certs/templates/rolebinding.yaml.

Copy link
Author

Choose a reason for hiding this comment

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

ash is the system shell used on busybox and local keyword are supported.

I'll look into this.

@math-nao
Copy link
Owner

Support for multi namespaces has been added in a previous version where it supports role and clusterrole.
I just released a new version and added some of your changes.
Could you please give a try to this new version and let me know?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants