Skip to content
This repository has been archived by the owner on Jun 10, 2024. It is now read-only.

Improve handling of mixed case #40

Open
planetf1 opened this issue May 3, 2022 · 0 comments
Open

Improve handling of mixed case #40

planetf1 opened this issue May 3, 2022 · 0 comments
Labels
containers documentation Improvements or additions to documentation enhancement New feature or request

Comments

@planetf1
Copy link
Member

planetf1 commented May 3, 2022

When the operator was originally developed, it was assumed the configmap names would exactly match the egeria server names.

However the former are restricted to be a valid DNS name - including being lowercase only - whilst the latter can be mixed case.

Currently the code mostly assumes they are the same until the filenames are needed, whereby it then introspects the configmap and restores the original case...

Need to look at

  • clarifying how case is handled in the docs
  • simplyfying the code - perhaps by keeping the server name & config name very distinct
@planetf1 planetf1 added documentation Improvements or additions to documentation enhancement New feature or request labels May 3, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
containers documentation Improvements or additions to documentation enhancement New feature or request
Projects
No open projects
Status: Todo
Development

No branches or pull requests

1 participant