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

adding tags to be returned as part of the tag_set data call #570

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

garkenxian
Copy link

#567

please be kind, this is my first foray into Go and working with Terraform providers. Thanks

Kenneth Garza and others added 2 commits October 27, 2023 13:20
updating tag_sets to return list of canonical tags that are assigned …
@@ -62,13 +84,44 @@ func getTagSetDataSchema() map[string]*schema.Schema {
}
}

func getTagSchemaForTagSet() map[string]*schema.Schema {
Copy link
Author

Choose a reason for hiding this comment

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

the getTagSchema() from schema_tag.go has a few extra values that is not needed here.

Copy link
Contributor

@zentron zentron left a comment

Choose a reason for hiding this comment

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

Thanks for this PR @eternalapprentice2000 .
Could you please update the TagSet Integration Test to validate that these values are returned.

In addition you may want to exclude all the non essential .devcontainer/ files from this PR.

I noticed one of the files you included contained an API Key, hopefully thats not copied from a production Octopus instance you have otherwise you may want to rotate it.

@@ -0,0 +1,14 @@
{
"name": "golang build env",
Copy link
Contributor

Choose a reason for hiding this comment

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

Looks like these devcontainer files were added to the commit, you may want to update the .gitignore

ACCEPT_EULA: "Y"
ADMIN_USERNAME: admin
ADMIN_PASSWORD: "P@ssw0rd!"
ADMIN_API_KEY: API-L2GV2ELTCZDRANT1OQO045GPZHXFGWR
Copy link
Contributor

Choose a reason for hiding this comment

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

If you copied this API key from a "real" octopus server you may want to revoke the key given its now appeared publicly

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