Work has started on new routes and the reference /cities spec, we should do it in one run on this bug.
Created attachment 128173 [details] [review] Bug 29620: Move the OpenAPI spec to YAML format This patch moves all the REST API spec pieces into YAML. To test: 1. Run: $ kshell k$ prove t/db_dependent/api/v1/* => SUCCESS: Tests pass 2. Apply this patch 3. Repeat 1 => SUCCESS: Tests still pass! 4. Sign off :-D Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Created attachment 128174 [details] [review] Bug 29620: (follow-up) Rename remaining old-style parameters The first routes we added named parameters in camelCase. We moved to snake_case really fast, but there are two that were kept by mistake. This patch fixes them. Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
If we agree with this, it should be pushed early this cycle. To allow people to produce their patches on top of it.
Created attachment 128211 [details] [review] Bug 29620: Move the OpenAPI spec to YAML format This patch moves all the REST API spec pieces into YAML. To test: 1. Run: $ kshell k$ prove t/db_dependent/api/v1/* => SUCCESS: Tests pass 2. Apply this patch 3. Repeat 1 => SUCCESS: Tests still pass! 4. Sign off :-D Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Created attachment 128212 [details] [review] Bug 29620: (follow-up) Rename remaining old-style parameters The first routes we added named parameters in camelCase. We moved to snake_case really fast, but there are two that were kept by mistake. This patch fixes them. Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Go Go Go... this is a no brainer to me.. all works, no breaking changes, just improves readability and makes it easier going forward. Going straight for QA.. Passed!
Pushed to master for 22.05, thanks to everybody involved [U+1F984]
(In reply to Fridolin Somers from comment #7) > Pushed to master for 22.05, thanks to everybody involved [U+1F984] Thank you. We need to have documentation people add more info now in Markdown
Pushed to 21.11.x for 21.11.01
I anticipate we may need to backport patches related to these files in 21.11.x branch. It makes sense to backport these changes for that reason.
Not backporting to 21.05 due to missing dependencies.