Skip to content

Ui/wrap tool ttl#9691

Merged
chelshaw merged 4 commits into
masterfrom
ui/wrap-tool-ttl
Aug 10, 2020
Merged

Ui/wrap tool ttl#9691
chelshaw merged 4 commits into
masterfrom
ui/wrap-tool-ttl

Conversation

@chelshaw
Copy link
Copy Markdown
Contributor

@chelshaw chelshaw commented Aug 7, 2020

This PR updates the TTL Picker in the Wrap Tool UI, fixing an issue where the wrap is created with the wrong TTL if the TTL value is not adjusted.

In Action:
wrap-tool-ttl

@chelshaw chelshaw added the ui label Aug 7, 2020
@chelshaw chelshaw added this to the 1.5.1 milestone Aug 7, 2020
@chelshaw chelshaw merged commit 111515b into master Aug 10, 2020
chelshaw added a commit that referenced this pull request Aug 11, 2020
* Update ttl picker in wrap tool
chelshaw added a commit that referenced this pull request Aug 17, 2020
* Update ttl picker in wrap tool
@briankassouf briankassouf deleted the ui/wrap-tool-ttl branch March 4, 2021 01:21
eualin pushed a commit to eualin/vault that referenced this pull request Sep 26, 2025
) (hashicorp#9698)

* actions(install-tools): include os and arch in cache key

When caching and/or restoring our tools we should include the os and
arch in the key to ensure that we don't accidentally download the wrong
tools on different runners.

We also update the nightlies to specifically cache arm64 before running
the tests.

* actionlint: add arm self-hosted runner keys

Signed-off-by: Ryan Cragun <me@ryan.ec>
Co-authored-by: Ryan Cragun <me@ryan.ec>
anwittin pushed a commit that referenced this pull request Sep 29, 2025
…) (#9712)

* actions(install-tools): include os and arch in cache key

When caching and/or restoring our tools we should include the os and
arch in the key to ensure that we don't accidentally download the wrong
tools on different runners.

We also update the nightlies to specifically cache arm64 before running
the tests.

* actionlint: add arm self-hosted runner keys

Signed-off-by: Ryan Cragun <me@ryan.ec>
Co-authored-by: Ryan Cragun <me@ryan.ec>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants