Skip to content

Conversation

@carlydf
Copy link
Collaborator

@carlydf carlydf commented Jul 31, 2025

What was changed and Why

We discussed using a . in the past, but the server won't be able to support that before the release of the controller. A / feels off if the ordering is resource/namespace because a resource is a sub-component of a namespace, and / implies file-system type hierarchy.

I had a convo with ChatGPT comparing some options just for fun, and I agree with it that an underscore separator with resource name followed by namespace name feels like the best option.

Checklist

  1. Closes

  2. How was this tested:
    Integration and unit tests

  3. Any docs updates needed?

@carlydf carlydf requested review from a team and jlegrone as code owners July 31, 2025 22:26
@Shivs11
Copy link
Member

Shivs11 commented Aug 1, 2025

can't seem to load the convo with our new best friend, so don't wanna preemptively approve this rn

@carlydf carlydf closed this Aug 1, 2025
@carlydf
Copy link
Collaborator Author

carlydf commented Aug 1, 2025

I changed my mind, I think resource/namespace is fine actually

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.

2 participants