update the existing secret comment for db secret #187
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: James Petersen <jpetersenames@gmail.com>
Signed-off-by: James Petersen <jpetersenames@gmail.com>
Btodhunter
pushed a commit
that referenced
this pull request
Feb 2, 2024
* add additional configs for 5.2.0 * updating helm unittests and readme to include changes * adding scratch for reports * update default of api delete keys to 365; add scratch volume to reports pod; add reports resources tests * adding additional reports resources test --------- Signed-off-by: Hung Nguyen <hung.tran.nguyen.585@gmail.com>
Btodhunter
pushed a commit
that referenced
this pull request
Feb 2, 2024
* add additional configs for 5.2.0 * updating helm unittests and readme to include changes * adding scratch for reports * update default of api delete keys to 365; add scratch volume to reports pod; add reports resources tests * adding additional reports resources test --------- Signed-off-by: Hung Nguyen <hung.tran.nguyen.585@gmail.com>
Btodhunter
added a commit
that referenced
this pull request
Feb 2, 2024
* Ent 5.2.0 updates (#187) * add additional configs for 5.2.0 * updating helm unittests and readme to include changes * adding scratch for reports * update default of api delete keys to 365; add scratch volume to reports pod; add reports resources tests * adding additional reports resources test * Enterprise split reports api (#188) * move reports service to seperate deployment * use enterprise prefix for helper templates * add rbacAuth container back to api pod * make reports-worker fullname match values key for the service * update unit tests * bump chart version * fix reports worker descriptions & alphabetize the top level key * add rbacAuth container to reports pod (#189) * add rbacAuth container to reports pod * add rbacAuth unit test for reports template * updating error message for image_ttl_days (#192) * Make scratch volume configurable (#193) * use helper template for scratch volume details. Allow component specific configs * add unit tests * add necessary values and update readme * bump chart version * bump enterprise image to v5.2.0 * update test snapshots * bump feeds chart version * bump chart version * update readme and values table * remove reports service from the ingress configuration --------- Signed-off-by: Brady Todhunter <bradyt@anchore.com> Signed-off-by: Hung Nguyen <hung.tran.nguyen.585@gmail.com> Co-authored-by: Hung Nguyen <hung.tran.nguyen.585@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Updated the comment for the feeds database existing secret to reflect that the ANCHORE_GITHUB_TOKEN is a required field if using the the
existingSecret.Signed-off-by: James Petersen jpetersenames@gmail.com