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

Improve configurability of registry and registry-console hosts #4181

Closed
wants to merge 3 commits into from

Conversation

dmsimard
Copy link
Contributor

This adds missing docs on the registry host route variable that can be specified to customize the host on the registry route.
Additionally, it adds the ability to customize the registry-console route host which allows to get something other than the provided default.

@openshift-bot
Copy link

Can one of the admins verify this patch?
I understand the following commands:

  • bot, add author to whitelist
  • bot, test pull request
  • bot, test pull request once

1 similar comment
@openshift-bot
Copy link

Can one of the admins verify this patch?
I understand the following commands:

  • bot, add author to whitelist
  • bot, test pull request
  • bot, test pull request once

@dmsimard
Copy link
Contributor Author

@abutcher as per discussed -- to improve the use case around standalone registry.

The SSL certificate bits are a bit more involved and I can't do it right now but it would definitely be something we'd need.

@abutcher abutcher self-assigned this May 15, 2017
@@ -55,6 +55,7 @@
- "{{ docker_registry_service_ip.results.clusterip }}"
- "{{ openshift_hosted_registry_name }}.default.svc"
- "{{ openshift_hosted_registry_name }}.default.svc.{{ openshift.common.dns_domain }}"
- "{{ openshift_hosted_registry_routehost | default(omit) }}"
Copy link
Member

Choose a reason for hiding this comment

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

omit fails in the negative case since we expect each item in this list to be a string. My proposed fix is to filter non-strings from the list, PR here #4197

David Moreau-Simard added 3 commits May 16, 2017 20:10
Otherwise, the registry-console is always defaulted to
registry-console-<namespace>-<host> and may not be customized
by the deployer.
openshift_hosted_registry_routehost allows a deployer to customize
the host for the docker-registry route.
Users can specify a custom hostname for the docker registry route.
We need to have the certificate match that host if it is specified
so that you can use docker commands against that host.
@dmsimard
Copy link
Contributor Author

Rebased my PR on top of master with #4197 which merged.

@abutcher
Copy link
Member

aos-ci-test

@openshift-bot
Copy link

success: "aos-ci-jenkins/OS_3.5_NOT_containerized, aos-ci-jenkins/OS_3.5_NOT_containerized_e2e_tests" for 6d32d6c (logs)

@openshift-bot
Copy link

success: "aos-ci-jenkins/OS_3.6_NOT_containerized, aos-ci-jenkins/OS_3.6_NOT_containerized_e2e_tests" for 6d32d6c (logs)

@openshift-bot
Copy link

success: "aos-ci-jenkins/OS_3.5_containerized, aos-ci-jenkins/OS_3.5_containerized_e2e_tests" for 6d32d6c (logs)

@openshift-bot
Copy link

success: "aos-ci-jenkins/OS_3.6_containerized, aos-ci-jenkins/OS_3.6_containerized_e2e_tests" for 6d32d6c (logs)

@openshift-bot
Copy link

error: "aos-ci-jenkins/OS_3.6_containerized, aos-ci-jenkins/OS_3.6_containerized_e2e_tests" for 6d32d6c (logs)

@dmsimard
Copy link
Contributor Author

If you wait a little bit, I'll rework this PR with SSL support for registry-console - it's a work in progress.

@abutcher
Copy link
Member

@dmsimard copy that

@openshift-bot
Copy link

success: "aos-ci-jenkins/OS_3.5_NOT_containerized, aos-ci-jenkins/OS_3.5_NOT_containerized_e2e_tests" for 6d32d6c (logs)

@openshift-bot
Copy link

success: "aos-ci-jenkins/OS_3.5_containerized, aos-ci-jenkins/OS_3.5_containerized_e2e_tests" for 6d32d6c (logs)

@openshift-bot
Copy link

success: "aos-ci-jenkins/OS_3.6_containerized, aos-ci-jenkins/OS_3.6_containerized_e2e_tests" for 6d32d6c (logs)

@openshift-bot
Copy link

error: "aos-ci-jenkins/OS_3.6_NOT_containerized, aos-ci-jenkins/OS_3.6_NOT_containerized_e2e_tests" for 6d32d6c (logs)

@dmsimard
Copy link
Contributor Author

Osboleted by #4256 .. going to close this.

@dmsimard dmsimard closed this May 21, 2017
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.

3 participants