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.

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

Add the ability to set seconds in cron schedule for testing purposes #22531

Merged
merged 24 commits into from
Aug 25, 2023

Conversation

Zlaticanin
Copy link
Contributor

No description provided.

fairclothjm and others added 19 commits August 22, 2023 12:36
* handle mutual exclusion of rotation fields

* fix import
* adv ttl mgmt: add rotation_window field

* do some rotation_window validation and add unit tests
#22341)

* general cleanup and refactor rotation type checks

* make NextRotationTime account for the rotation type

* add comments
* add unit tests to handle mutual exclusion

* revert rotation_test.go and add missing test case to path_roles_test.go
* support manual rotation for schedule based roles

* update description and naming
* consider rotation window

ensure rotations only occur within a rotation window for schedule-based
rotations

* use helper method to set priority in rotateCredential

* fix bug with priority check

* remove test for now

* add and remove comments
* update read static-creds endpoint to include correct resp data

* return rotation_window if set

* update
@Zlaticanin Zlaticanin requested a review from a team as a code owner August 23, 2023 19:46
@Zlaticanin Zlaticanin removed the request for review from a team August 23, 2023 19:46
@github-actions github-actions bot added the hashicorp-contributed-pr If the PR is HashiCorp (i.e. not-community) contributed label Aug 23, 2023
@Zlaticanin Zlaticanin requested a review from fairclothjm August 23, 2023 19:46
@github-actions
Copy link

Build Results:
All builds succeeded! ✅

@github-actions
Copy link

github-actions bot commented Aug 23, 2023

CI Results:
All Go tests succeeded! ✅

Base automatically changed from VAULT-17232/adv-ttl-mgmt to main August 24, 2023 21:45
@fairclothjm fairclothjm requested a review from a team as a code owner August 24, 2023 21:45
@Zlaticanin Zlaticanin removed the request for review from a team August 24, 2023 22:31
@Zlaticanin Zlaticanin requested a review from a team August 25, 2023 00:14
@Zlaticanin
Copy link
Contributor Author

Ugh, I had some issues with merging main into this PR since we merged the feature branch. I tried rebasing first, but I was having issues with go.mod and go.sum, not sure why.

@Zlaticanin Zlaticanin requested a review from fairclothjm August 25, 2023 16:15
@Zlaticanin Zlaticanin merged commit 2d0d5c7 into main Aug 25, 2023
@Zlaticanin Zlaticanin deleted the VAULT-19131/toggle-seconds branch August 25, 2023 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hashicorp-contributed-pr If the PR is HashiCorp (i.e. not-community) contributed pr/no-changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants