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

Extra tags #169

Merged
merged 4 commits into from
May 16, 2024
Merged

Extra tags #169

merged 4 commits into from
May 16, 2024

Conversation

awick
Copy link
Contributor

@awick awick commented May 16, 2024

Description

Add the ability to assign extra tags to all resources and add the arkime_cluster tag to all resources. Also removed the old way tags were being added to the OS Domain and moved to the cdk-lib

Testing

  • Tested creating a cluster without tags, only arkime_cluster shows up
  • Tested rerunning with tags on that same cluster, now the new tags show up
  • Tested creating a new cluster with tags, all tags showed up

License

I confirm that this contribution is made under an Apache 2.0 license and that I have the authority necessary to make this contribution on behalf of its copyright owner.

awick added 3 commits May 15, 2024 15:59
We now tag all resources with the arkime_cluster tag.
Can also use --extra-tag <key> <value> to have more tags added to all
resources
cdk-lib/cloud-demo.ts Show resolved Hide resolved
@awick awick merged commit 04a4732 into main May 16, 2024
5 checks passed
@awick awick deleted the extra-tags branch May 16, 2024 14:31
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.

3 participants