Good practices - updating state upon retrying about kopf HOT 6 CLOSED

I'm probably missing something, but why not just do this?

@kopf.on.update('httprequests', retries=3, backoff=10)
def handler(memo: kopf.Memo, spec, name, namespace, logger, **kwargs):
    r = requests.request(some_url)
    if r.status_code == 500:
        raise kopf.TemporaryError(f"Resource at {some_url} returns {r.status_code}", delay=60)

from kopf.

That does not update status field.

from kopf.

There is the patch kwarg. You can populate it with whatever you want during the handler execution. Kopf will add a few things of its own later after the error (or success) — and will apply it to the k8s api.

Manually patching the resource is also a right way to go, there is nothing wrong with it (except for one extremely rare edge case under high load, which will be fixed soon). The only downside is that you will have 2 patch api calls: one from you, one from kopf; and you will also have to authenticate somehow not with kopf’s login machinery.

from kopf.

@nolar
if I read the docs correctly https://kopf.readthedocs.io/en/stable/kwargs/#patching patch is a pass through object that I can fill with anything that will get passed to the patching logic

@kopf.on.update('httprequests', retries=3, backoff=10)
def handler(memo: kopf.Memo, spec, name, namespace, logger, **kwargs):
    r = requests.request(some_url)
    if r.status_code == 500:
       kwargs['patch'] = {
            'status': {
                'handler': {
                    'httpStatusCode': 500
                }
            }
        }
        raise kopf.TemporaryError(f"Resource at {some_url} returns {r.status_code}", delay=60)

is it the right way to do? I am asking, because following snippet does not work.

from kopf.

The right way. But in your case, you override the kwarg key in the local dict, you do not modify the object itself. Try this:

def …(…, patch, **kwargs):
    patch.status.setdefault('handler', {})['httpStatusCode'] = 500

from kopf.

Perfect, it works.

Thanks for the guidance.

Closing/

from kopf.