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

Add draft for KaaS LB standard (#169) #648

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

cah-hbaum
Copy link
Contributor

Follow-Up PR to #169
Everything still relevant will be discussed in here now.

* Add draft for KaaS LB standard

Signed-off-by: Joshua Mühlfort <[email protected]>

* Rework large chunks of the record

Signed-off-by: Joshua Mühlfort <[email protected]>

* Add Decision section again; Fix typo etc.

Signed-off-by: Joshua Mühlfort <[email protected]>

* Fix conformance tests wording

Signed-off-by: Joshua Mühlfort <[email protected]>

* Fix strong wording

Signed-off-by: Joshua Mühlfort <[email protected]>

* Add section about OpenStacks CCM defaults

Signed-off-by: Joshua Mühlfort <[email protected]>

* Change wording

Signed-off-by: Joshua Mühlfort <[email protected]>

* Requests->Connections

Signed-off-by: Joshua Mühlfort <[email protected]>

* Add ref implementation hint

Signed-off-by: Joshua Mühlfort <[email protected]>

* Make decision items more conformant to each other

Signed-off-by: Joshua Mühlfort <[email protected]>

* Minor wording changes/additions

Signed-off-by: Joshua Mühlfort <[email protected]>

* Reword logging/...

Signed-off-by: Joshua Mühlfort <[email protected]>

* Clarify Option 1

Signed-off-by: Joshua Mühlfort <[email protected]>

* Improve wording

Signed-off-by: Joshua Mühlfort <[email protected]>

* Add another option

Signed-off-by: Joshua Mühlfort <[email protected]>

* More clear wording in options

Signed-off-by: Joshua Mühlfort <[email protected]>

* Address review

Signed-off-by: Joshua Mühlfort <[email protected]>

* Treat PROXY protocol and HTTP headers as options of the same category

Signed-off-by: Joshua Mühlfort <[email protected]>

* Fix indentation

Signed-off-by: Joshua Mühlfort <[email protected]>

* Select option regarding "Local" policy

Signed-off-by: Joshua Mühlfort <[email protected]>

---------

Signed-off-by: Joshua Mühlfort <[email protected]>
@cah-hbaum cah-hbaum added Container Issues or pull requests relevant for Team 2: Container Infra and Tooling standards Issues / ADR / pull requests relevant for standardization & certification SCS is standardized SCS is standardized labels Jun 26, 2024
Small grammar adjustments and fixes.

Signed-off-by: Hannes Baum <[email protected]>
Fixes to please the linter.

Signed-off-by: Hannes Baum <[email protected]>
@joshmue
Copy link
Contributor

joshmue commented Aug 28, 2024

I recall that the type of the record was the topic of discussion in #169 - whether this should be a "Decision Record" or "Standard".

I guess that changing the type in the header section to "Standard" would already bring this document mostly in line with other standard documents like #658 - regarding formal aspects as well as value for CSP's/users.

@anjastrunk anjastrunk mentioned this pull request Sep 4, 2024
29 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Container Issues or pull requests relevant for Team 2: Container Infra and Tooling SCS is standardized SCS is standardized standards Issues / ADR / pull requests relevant for standardization & certification
Projects
Status: Backlog
Development

Successfully merging this pull request may close these issues.

2 participants