[PR] Optionally remove the Status handler_id prefix

[kopf-archiver]

A pull request by lack at 2020-03-02 21:48:59+00:00
Original URL: zalando-incubator/kopf#320
 

What do these changes do?

As proposed in issue #319, this introduces an optional status_prefix boolean that can be added to any resource handler definition. If set to True (or omitted), status returned by the handler is still placed beneath the handler_id in the resulting object's status subresource. If set to False, the status returned by the handler is use at the top-level of the status subresource.

Description

Fully implements the proposal in issue #319.

Issues/PRs

Issues: #319

Type of changes

• New feature (non-breaking change which adds functionality)

Checklist

• The code addresses only the mentioned problem, and this problem only
• I think the code is well written
• Unit tests for the changes exist
• Documentation reflects the changes
• If you provide code modification, please add yourself to CONTRIBUTORS.txt

---

Commented by ps-jay at 2020-08-20 05:16:45+00:00
 

This would be a really nice feature to have

---

Commented by philipp-sontag-by at 2020-08-22 11:27:17+00:00
 

Hey lack,
is there any chance you would be willing to update this branch and move the PR to the new repository at https://github.com/nolar/kopf?

If you don't have time at the moment would it be ok if I pick this up based on the changes you already made?

---

Commented by lack at 2020-08-22 19:10:46+00:00
 

philipp-sontag-by : I'll try to redo this in the other repo, but I may not have time in the near future (starting a new job this week). I'd also be happy for you to carry this forward if you have time. Let me know here if you've got it in another PR so we don't duplicate effort.

[ps-jay]

I'm keen for this PR to be considered

[psontag]

@nolar would you accept a contribution for this issue?

[samgiles]

I've ported the original PR in #579!

[nolar]

Closed together with #579 — see the reasoning in #579 (comment)