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.

Sign up for GitHub

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

Jump to bottom

feat: secret owners do not auto-peek, and can use refresh #1067

Merged
merged 9 commits into from
Nov 23, 2023
Merged

feat: secret owners do not auto-peek, and can use refresh #1067

merged 9 commits into from
Nov 23, 2023

Commits on Nov 19, 2023

  1. Reflect upcoming Juju secret behaviour. 

    The special-casing for secret-get is going to be removed, so that owners have the same behaviour as everyone else - they get the tracked revision of the secret, and can call peek to get the latest content, or refresh to get the latest content and update the tracking pointer.
    @tonyandrewmeyer
    tonyandrewmeyer committed Nov 19, 2023
    Configuration menu
    Copy the full SHA
    d02f829 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'main' into secret-owner-behaviour-change-1062

    @tonyandrewmeyer
    tonyandrewmeyer authored Nov 19, 2023
    Configuration menu
    Copy the full SHA
    5ff5d4e View commit details
    Browse the repository at this point in the history

Commits on Nov 20, 2023

  1. Add additional secret_get tests.

    @tonyandrewmeyer
    tonyandrewmeyer committed Nov 20, 2023
    Configuration menu
    Copy the full SHA
    1bc98b0 View commit details
    Browse the repository at this point in the history

Commits on Nov 22, 2023

  1. Merge branch 'main' into secret-owner-behaviour-change-1062

    @tonyandrewmeyer
    tonyandrewmeyer authored Nov 22, 2023
    Configuration menu
    Copy the full SHA
    ba3d43d View commit details
    Browse the repository at this point in the history

Commits on Nov 23, 2023

  1. Remove note: true now, not true after Juju updates. 

    It seems to make sense to remove this at the same time we're changing the Harness behaviour, even though we're a bit ahead of Juju.
    @tonyandrewmeyer
    tonyandrewmeyer committed Nov 23, 2023
    Configuration menu
    Copy the full SHA
    d5a9d0b View commit details
    Browse the repository at this point in the history

  2. Merge branch 'main' into secret-owner-behaviour-change-1062

    @benhoyt
    benhoyt authored Nov 23, 2023
    Configuration menu
    Copy the full SHA
    55419f8 View commit details
    Browse the repository at this point in the history

  3. Set leader where it's (now) expected.

    @tonyandrewmeyer
    tonyandrewmeyer committed Nov 23, 2023
    Configuration menu
    Copy the full SHA
    75ee3bb View commit details
    Browse the repository at this point in the history

  4. Merge branch 'main' into secret-owner-behaviour-change-1062

    @benhoyt
    benhoyt authored Nov 23, 2023
    Configuration menu
    Copy the full SHA
    3ca86bc View commit details
    Browse the repository at this point in the history

  5. Merge branch 'main' into secret-owner-behaviour-change-1062

    @benhoyt
    benhoyt authored Nov 23, 2023
    Configuration menu
    Copy the full SHA
    9679370 View commit details
    Browse the repository at this point in the history