Skip to content

fix doc: metadata.annotations keys should not allow spaces#5013

Open
peterhaochen47 wants to merge 1 commit intocloudfoundry:mainfrom
peterhaochen47:pr/main/fix-doc-label-key-april926
Open

fix doc: metadata.annotations keys should not allow spaces#5013
peterhaochen47 wants to merge 1 commit intocloudfoundry:mainfrom
peterhaochen47:pr/main/fix-doc-label-key-april926

Conversation

@peterhaochen47
Copy link
Copy Markdown
Member

@peterhaochen47 peterhaochen47 commented Apr 9, 2026

  • the validation logic in app/messages/metadata_validator_helper.rb does not allow spaces in keys
  • added a relevant test case
  • fixed the examples provided in doc

Thanks for contributing to cloud_controller_ng. To speed up the process of reviewing your pull request please provide us with:

  • A short explanation of the proposed change:

  • An explanation of the use cases your change solves

  • Links to any other associated PRs

  • I have reviewed the contributing guide

  • I have viewed, signed, and submitted the Contributor License Agreement

  • I have made this pull request to the main branch

  • I have run all the unit tests using bundle exec rake

  • I have run CF Acceptance Tests

@peterhaochen47 peterhaochen47 changed the title fix doc: metadata.label keys should not allow spaces fix doc: metadata.annotations keys should not allow spaces Apr 10, 2026
@peterhaochen47 peterhaochen47 force-pushed the pr/main/fix-doc-label-key-april926 branch from 5b4f97a to 82fb28d Compare April 10, 2026 19:18
- the validation logic in app/messages/metadata_validator_helper.rb
does not allow spaces in keys
- added a relevant test case
- fixed the examples provided in doc
@peterhaochen47 peterhaochen47 force-pushed the pr/main/fix-doc-label-key-april926 branch from 82fb28d to 5c3080d Compare April 10, 2026 19:23
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.

1 participant