Christopher Swenson
961bf20bdb
Use enumer to generate String() methods for most enums ( #25705 )
...
We have many hand-written String() methods (and similar) for enums.
These require more maintenance and are more error-prone than using
automatically generated methods. In addition, the auto-generated
versions can be more efficient.
Here, we switch to using https://github.com/loggerhead/enumer , itself
a fork of https://github.com/diegostamigni/enumer , no longer maintained,
and a fork of the mostly standard tool
https://pkg.go.dev/golang.org/x/tools/cmd/stringer .
We use this fork of enumer for Go 1.20+ compatibility and because
we require the `-transform` flag to be able to generate
constants that match our current code base.
Some enums were not targeted for this change:
2024-04-17 11:14:14 -07:00
divyaac
3132592c19
Added exponential backoff ( #25497 )
...
* Applied Patch
* Added changelog
* Edited changelog
* Added constants to be shared
* Edited changelog verbage
* Removed copy and paste error
* Moved the constants
* Fixed static checks
2024-02-20 11:42:59 -08:00
Josh Black
fa13dbd381
add gosimport to make fmt and run it ( #25383 )
...
* add gosimport to make fmt and run it
* move installation to tools.sh
* correct weird spacing issue
* Update Makefile
Co-authored-by: Nick Cabatoff <ncabatoff@hashicorp.com >
* fix a weird issue
---------
Co-authored-by: Nick Cabatoff <ncabatoff@hashicorp.com >
2024-02-13 14:07:02 -08:00
Daniel Huckins
d1e1abd2c7
VAULT-19681 allow users to specify files for agent child process stdout/stderr ( #22812 )
...
* allow users to specify files for child process stdout/stderr
* added changelog
* check if exec config is nil
* fix test
* first attempt at a test
* revise test
* passing test
* added failing test
* Apply suggestions from code review
Co-authored-by: Anton Averchenkov <84287187+averche@users.noreply.github.com >
* code review suggestions
* always close log files
* refactor to use real files
* hopefully fixed tests
* add back bool gates so we don't close global stdout/stderr
* compare to os.Stdout/os.Stderr
* remove unused
---------
Co-authored-by: Anton Averchenkov <84287187+averche@users.noreply.github.com >
2023-09-12 13:54:37 -04:00
hashicorp-copywrite[bot]
0b12cdcfd1
[COMPLIANCE] License changes ( #22290 )
...
* Adding explicit MPL license for sub-package.
This directory and its subdirectories (packages) contain files licensed with the MPLv2 `LICENSE` file in this directory and are intentionally licensed separately from the BSL `LICENSE` file at the root of this repository.
* Adding explicit MPL license for sub-package.
This directory and its subdirectories (packages) contain files licensed with the MPLv2 `LICENSE` file in this directory and are intentionally licensed separately from the BSL `LICENSE` file at the root of this repository.
* Updating the license from MPL to Business Source License.
Going forward, this project will be licensed under the Business Source License v1.1. Please see our blog post for more details at https://hashi.co/bsl-blog , FAQ at www.hashicorp.com/licensing-faq, and details of the license at www.hashicorp.com/bsl.
* add missing license headers
* Update copyright file headers to BUS-1.1
* Fix test that expected exact offset on hcl file
---------
Co-authored-by: hashicorp-copywrite[bot] <110428419+hashicorp-copywrite[bot]@users.noreply.github.com>
Co-authored-by: Sarah Thompson <sthompson@hashicorp.com >
Co-authored-by: Brian Kassouf <bkassouf@hashicorp.com >
2023-08-10 18:14:03 -07:00
Anton Averchenkov
bc9a39a2f1
agent: Fix bug with early exits during restarts ( #20950 )
2023-06-02 09:05:13 -04:00
Daniel Huckins
a5a49cde3f
agent: Prevent multiple restarts of child process in supervisor mode ( #20940 )
...
* try timer
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* add to config
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* add test to verify behavior
* remove closer func -- it was causing a race condition
* Revert "add to config"
This reverts commit 1de6af0ff676029e290cc42a0bb2b7e6f597d1a6.
* rename variables, add comment
* comment
* comment re debounce timer
* don't skip tests
* fix comment
* formatting
* formatting
---------
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
Co-authored-by: Anton Averchenkov <anton.averchenkov@hashicorp.com >
2023-06-01 19:55:50 -04:00
Anton Averchenkov
9be2903a34
agent: Don't restart process unless environment variables changed ( #20917 )
2023-06-01 10:57:45 -04:00
Anton Averchenkov
fe53c4684c
agent: Don't render templates to stdout in supervisor mode ( #20884 )
2023-05-31 12:04:33 -04:00
Anton Averchenkov
7f2d3f2c5c
fix exitCh race condition ( #20817 )
2023-05-30 12:23:51 -04:00
Daniel Huckins
a66074425d
agent: Add implementation for injecting secrets as environment variables to vault agent cmd ( #20739 )
...
* added exec and env_template config/parsing
* add tests
* we can reuse ctconfig here
* do not create a non-nil map
* check defaults
* Apply suggestions from code review
Co-authored-by: Anton Averchenkov <84287187+averche@users.noreply.github.com >
* Apply suggestions from code review
Co-authored-by: Anton Averchenkov <84287187+averche@users.noreply.github.com >
* first go of exec server
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* convert to list
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* convert to list
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* sig test
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* add failing example
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* refactor for config changes
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* add test for invalid signal
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* account for auth token changes
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* only start the runner once we have a token
* tests in diff branch
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* Apply suggestions from code review
Co-authored-by: Anton Averchenkov <84287187+averche@users.noreply.github.com >
* fix rename
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* Update command/agent/exec/exec.go
Co-authored-by: Anton Averchenkov <84287187+averche@users.noreply.github.com >
* apply suggestions from code review
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* cleanup
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* remove unnecessary lock
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* refactor to use enum
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* dont block
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* handle default
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* make more explicit
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* cleanup
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* remove unused
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* remove unused file
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* remove test app
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* Apply suggestions from code review
Co-authored-by: Anton Averchenkov <84287187+averche@users.noreply.github.com >
* apply suggestions from code review
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* update comment
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* add changelog
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* new channel for exec server token
* wire to run with vault agent
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* watch for child process to exit on its own
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* block before returning
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
---------
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
Co-authored-by: Anton Averchenkov <84287187+averche@users.noreply.github.com >
2023-05-25 09:23:56 -04:00
Daniel Huckins
17a1e78ffb
agent: Add implementation for injecting secrets as environment variables ( #20628 )
...
* added exec and env_template config/parsing
* add tests
* we can reuse ctconfig here
* do not create a non-nil map
* check defaults
* Apply suggestions from code review
Co-authored-by: Anton Averchenkov <84287187+averche@users.noreply.github.com >
* Apply suggestions from code review
Co-authored-by: Anton Averchenkov <84287187+averche@users.noreply.github.com >
* first go of exec server
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* convert to list
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* convert to list
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* sig test
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* add failing example
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* refactor for config changes
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* add test for invalid signal
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* account for auth token changes
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* only start the runner once we have a token
* tests in diff branch
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* Apply suggestions from code review
Co-authored-by: Anton Averchenkov <84287187+averche@users.noreply.github.com >
* fix rename
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* Update command/agent/exec/exec.go
Co-authored-by: Anton Averchenkov <84287187+averche@users.noreply.github.com >
* apply suggestions from code review
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* cleanup
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* remove unnecessary lock
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* refactor to use enum
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* dont block
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* handle default
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* make more explicit
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* cleanup
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* remove unused
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* remove unused file
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* remove test app
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* Apply suggestions from code review
Co-authored-by: Anton Averchenkov <84287187+averche@users.noreply.github.com >
* apply suggestions from code review
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* update comment
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* add changelog
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
* watch for child process to exit on its own
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
---------
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com >
Co-authored-by: Anton Averchenkov <84287187+averche@users.noreply.github.com >
2023-05-24 16:56:06 -04:00