Skip to content

Adding a Tags - key:value pair object to the schema #1207

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

Merged
merged 7 commits into from
Oct 17, 2024

Conversation

floydtree
Copy link
Contributor

@floydtree floydtree commented Oct 16, 2024

Related Issue: From Oct 16th, 2024 Network Activity Call.

Currently, we have labels, a string array attribute that can be used to populate labels. A String array is not sufficient when labels come in the form key:value pairs. Therefore, adding a tag object which can allow users to populate key:value styled labels/tags in the events.

Description of changes:

  1. Adding a tag object, adding tags to dictionary.
  2. Deprecating an inconsistently used tag attribute, captioned Image Tag (was used in Image and Container) adding labels and tags to the object as alternatives and for consistency.
  3. Adding tags to all the objects where, labels is currently used. And sorting the file contents.
  4. Sorting dictionary.
  5. Minor description adjustments.

Signed-off-by: Rajas Panat <rajaspa@amazon.com>
Signed-off-by: Rajas Panat <rajaspa@amazon.com>
Signed-off-by: Rajas Panat <rajaspa@amazon.com>
Signed-off-by: Rajas Panat <rajaspa@amazon.com>
@floydtree floydtree self-assigned this Oct 16, 2024
@floydtree floydtree added enhancement New feature or request non_breaking Non Breaking, backwards compatible changes v1.4.0 Changes marked for the upcoming version 1.4.0 labels Oct 16, 2024
Signed-off-by: Rajas Panat <rajaspa@amazon.com>
@floydtree floydtree marked this pull request as ready for review October 16, 2024 19:37
Copy link
Contributor

@jonrau-at-queryai jonrau-at-queryai left a comment

Choose a reason for hiding this comment

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

Nice changes, well summarized. Now we can answer the age old question from people who use K8s or the wrong cloud (😉 ) about how to use tag.

Copy link
Contributor

@zschmerber zschmerber left a comment

Choose a reason for hiding this comment

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

Cool LGTM, who is going to update the AWS markdown examples mapping now?

@floydtree
Copy link
Contributor Author

floydtree commented Oct 17, 2024

Cool LGTM, who is going to update the AWS markdown examples mapping now?

I like your enthusiasm. If you notice, all the AWS mappings are versioned, we'll update it as we release the mappings for the next stable release.

Copy link
Contributor

@Aniak5 Aniak5 left a comment

Choose a reason for hiding this comment

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

LGTM

@floydtree floydtree merged commit 5998683 into ocsf:main Oct 17, 2024
3 checks passed
@floydtree floydtree deleted the key_value_pair_object branch October 17, 2024 19:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request non_breaking Non Breaking, backwards compatible changes v1.4.0 Changes marked for the upcoming version 1.4.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants