Skip to content

Conversation

@jremy42
Copy link
Contributor

@jremy42 jremy42 commented Dec 2, 2025

close #3337

@codecov-commenter
Copy link

codecov-commenter commented Dec 2, 2025

⚠️ JUnit XML file not found

The CLI was unable to find any JUnit XML files to upload.
For more help, visit our troubleshooting guide.

@jremy42 jremy42 force-pushed the feat/mongodb-instance-snapshot-action branch 17 times, most recently from 263c8dd to 8432199 Compare December 3, 2025 15:36
@jremy42 jremy42 marked this pull request as ready for review December 3, 2025 17:27
@jremy42 jremy42 requested a review from remyleone as a code owner December 3, 2025 17:27
Required: true,
Description: "MongoDB instance ID to snapshot. Can be a plain UUID or a regional ID.",
Validators: []validator.String{
stringvalidator.LengthAtLeast(1),
Copy link
Member

Choose a reason for hiding this comment

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

I don't think this validator is very helpful. It should be either removed or replaced with a proper validator that can check for a localized/unlocalized id


snapshotName := data.Name.ValueString()
if snapshotName == "" {
snapshotName = "tf-mongodb-snapshot"
Copy link
Member

Choose a reason for hiding this comment

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

could you add action also here to have more context in this default value?

@jremy42 jremy42 force-pushed the feat/mongodb-instance-snapshot-action branch from da5f24c to 6075b7d Compare December 11, 2025 14:02
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.

Add support for actions in mongodb

4 participants