Dima Tisnek
Dima Tisnek
In term of logic the above is actually correct (this unit **can't manage** granted secrets). In term of the error message though, this is confusing, if None really means granted.
quick fix: https://github.com/dimaqq/operator/pull/72 I want to think a bit more about a more comprehensive fix... maybe.
@copilot there's no need for the new test; existing tests already cover this. the exception text should be `this secret is not owned by this unit/app. Did you forget passing...
ref: https://github.com/canonical/operator/pull/440 there's a test in `test/test_main.py` as well.
I wonder about the overall tone of this do. There's a lot of "you this" and "you that", and now we're adding "you shouldn't". How about something like this: Ideally...
I've skimmed through our collection of charms and it seems that most common use is: `foo = container.pull(...).read()` With a few variations like `foo = reversed(container.pull(...).readlines())`. Perhaps our API doesn't...
I'm running integration tests on `main` vs `3.3.1` here to make sure it's not a regression: https://github.com/canonical/operator/actions/workflows/integration.yaml
main passed, 3.3.1 failed. I guess this means that the test is merely flaky.
Still flaky: - failed https://github.com/canonical/operator/actions/runs/18840693298/job/53752043204 - passed https://github.com/canonical/operator/actions/runs/18840693298/job/53753970580
Noticed while working on #1988