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

PRODENG-2237 Added pruning policy resource #24

Merged
merged 2 commits into from
Sep 14, 2023
Merged

Conversation

cranzy
Copy link
Collaborator

@cranzy cranzy commented Sep 12, 2023

Description

  1. Added pruning policy resource
  2. Switched the deprecated rand.Seed(time.Now().UnixNano()) to rand.New(rand.NewSource(time.Now().UnixNano()))
  3. Updated the golang dependencies

Issue

https://mirantis.jira.com/browse/PRODENG-2237

@cranzy cranzy changed the title PRODENG-2237 Added pruning policy resource [WIP]PRODENG-2237 Added pruning policy resource Sep 12, 2023
@cranzy cranzy force-pushed the 2237-pruning-policy branch from 24b0fc9 to 50e27df Compare September 12, 2023 14:51
Copy link
Collaborator

@james-nesbitt james-nesbitt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

as you are doing a go module bump, you should also consider updating the versions of the critical imports, such as the terraform-plugin-framework.

Also, can you try to update the golangci and goreleaser GH action versions, for which there are updates as reported by dependabot.

internal/client/utils.go Show resolved Hide resolved
@cranzy cranzy force-pushed the 2237-pruning-policy branch from 50e27df to bbf5d0d Compare September 13, 2023 14:21
@cranzy cranzy force-pushed the 2237-pruning-policy branch from bbf5d0d to 13868b6 Compare September 14, 2023 07:32
@cranzy cranzy changed the title [WIP]PRODENG-2237 Added pruning policy resource PRODENG-2237 Added pruning policy resource Sep 14, 2023
@cranzy cranzy force-pushed the 2237-pruning-policy branch from 13868b6 to 2698e9e Compare September 14, 2023 10:52
@cranzy cranzy force-pushed the 2237-pruning-policy branch 3 times, most recently from ee8bc04 to f113778 Compare September 14, 2023 11:41
@cranzy cranzy force-pushed the 2237-pruning-policy branch from f113778 to 21c38cf Compare September 14, 2023 12:04
@cranzy cranzy merged commit de86d4d into main Sep 14, 2023
12 checks passed
@cranzy cranzy deleted the 2237-pruning-policy branch September 14, 2023 12:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants