Skip to content

Vault 936: use core.activeContext in ActivityLog#13083

Merged
swayne275 merged 7 commits intomainfrom
vault-936
Nov 9, 2021
Merged

Vault 936: use core.activeContext in ActivityLog#13083
swayne275 merged 7 commits intomainfrom
vault-936

Conversation

@swayne275
Copy link
Copy Markdown
Contributor

No description provided.

Comment thread vault/activity_log.go Outdated
@vercel vercel bot temporarily deployed to Preview – vault-storybook November 8, 2021 22:48 Inactive
@vercel vercel bot temporarily deployed to Preview – vault November 8, 2021 22:48 Inactive
@swayne275 swayne275 requested a review from ncabatoff November 8, 2021 22:48
@vercel vercel bot temporarily deployed to Preview – vault-storybook November 8, 2021 22:50 Inactive
@vercel vercel bot temporarily deployed to Preview – vault November 8, 2021 22:50 Inactive
@swayne275
Copy link
Copy Markdown
Contributor Author

i'm not sure if there's any value in updating the context used by tests. and if i do, i might want to have a helper to grab the context under stateLock

thoughts @ncabatoff ?

Comment thread vault/activity_log.go Outdated
@ncabatoff
Copy link
Copy Markdown
Collaborator

i'm not sure if there's any value in updating the context used by tests. and if i do, i might want to have a helper to grab the context under stateLock

Let's not worry about the tests here unless they become a problem.

@vercel vercel bot temporarily deployed to Preview – vault November 9, 2021 17:01 Inactive
@vercel vercel bot temporarily deployed to Preview – vault-storybook November 9, 2021 17:01 Inactive
@vercel vercel bot temporarily deployed to Preview – vault November 9, 2021 17:03 Inactive
@vercel vercel bot temporarily deployed to Preview – vault-storybook November 9, 2021 17:03 Inactive
@swayne275 swayne275 requested a review from ncabatoff November 9, 2021 17:04
@swayne275 swayne275 added this to the 1.7.7 milestone Nov 9, 2021
@vercel vercel bot temporarily deployed to Preview – vault-storybook November 9, 2021 18:06 Inactive
@vercel vercel bot temporarily deployed to Preview – vault November 9, 2021 18:06 Inactive
@swayne275 swayne275 merged commit 97610c1 into main Nov 9, 2021
@swayne275 swayne275 deleted the vault-936 branch November 9, 2021 18:47
swayne275 added a commit that referenced this pull request Nov 9, 2021
* update activity log to use core's activeContext for cleaner worker termination

* update tests to use core activeContext instead of generic context

* pass context around instead

* revert context change

* undo test context changes

* change worker context

* accidentally undid context for fcn signature changes
swayne275 added a commit that referenced this pull request Nov 9, 2021
* update activity log to use core's activeContext for cleaner worker termination

* update tests to use core activeContext instead of generic context

* pass context around instead

* revert context change

* undo test context changes

* change worker context

* accidentally undid context for fcn signature changes
swayne275 added a commit that referenced this pull request Nov 9, 2021
* update activity log to use core's activeContext for cleaner worker termination

* update tests to use core activeContext instead of generic context

* pass context around instead

* revert context change

* undo test context changes

* change worker context

* accidentally undid context for fcn signature changes
swayne275 added a commit that referenced this pull request Nov 9, 2021
* update activity log to use core's activeContext for cleaner worker termination

* update tests to use core activeContext instead of generic context

* pass context around instead

* revert context change

* undo test context changes

* change worker context

* accidentally undid context for fcn signature changes
swayne275 added a commit that referenced this pull request Nov 9, 2021
* update activity log to use core's activeContext for cleaner worker termination

* update tests to use core activeContext instead of generic context

* pass context around instead

* revert context change

* undo test context changes

* change worker context

* accidentally undid context for fcn signature changes
swayne275 added a commit that referenced this pull request Nov 9, 2021
* update activity log to use core's activeContext for cleaner worker termination

* update tests to use core activeContext instead of generic context

* pass context around instead

* revert context change

* undo test context changes

* change worker context

* accidentally undid context for fcn signature changes
pbromb pushed a commit to pbromb/vault that referenced this pull request Apr 13, 2026
…ontainers (hashicorp#12920) (hashicorp#13083) (hashicorp#13161)

* update user directive for dockerfile

run as vault user by default

* add changelog

* also set the user to vault in ent dockerfile

* update changelog

* add the user directive in the Dockerfile of the repo root

it is the one used for production builds

* remove user directive from build stage, add user directive to alpine stage

Co-authored-by: mickael-hc <86245626+mickael-hc@users.noreply.github.com>
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants