Skip to content

[Rule Tuning] Bumping min-stack version for Google Workspace to 8.4#2467

Merged
terrancedejesus merged 2 commits intomainfrom
google-workspace-min-stack-rules-8.4
Jan 13, 2023
Merged

[Rule Tuning] Bumping min-stack version for Google Workspace to 8.4#2467
terrancedejesus merged 2 commits intomainfrom
google-workspace-min-stack-rules-8.4

Conversation

@terrancedejesus
Copy link
Contributor

@terrancedejesus terrancedejesus commented Jan 13, 2023

Issues

#2464
elastic/integrations#3430

Summary

While updating and locking rule versions for the v8.3.4 and v8.4.2 release, we noticed that Google Workspace rule versions were double bumped. Analysis revealed this to be a result of build-time fields, specifically related_integrations. At the Detection Rules 8.3 branch, the build-time version value for related_integrations is 1.2.0, whereas for the 8.4 branch the value is 2.0.0 and thus the versions double bumped.

We have had a discussion with @spong about potentially adjusting this build-time value being determined via Fleet instead as we would supply the package and integration. While this is a long-term solution, a stop-gap solution is necessary to ensure we do not continue to double bump these versions, but continue our release process.

Bumping the min-stack rule to 8.4 would fork the Google Workspace rules and thus prior to 8.4 stacks they would receive the 1.2.0 package whereas 8.4+ would receive the 2.0.0 package. In regards to versioning, this would allow the rule to be in a diverged state and thus the SHA256 sum remain the same for the 8.3 forked version and 8.4+ separately.

Additionally, the Google Workspace integration received an update which may confirm the changes were necessary to change the Kibana stack version to 8.4.0.

Screen Shot 2023-01-13 at 1 10 02 PM

Copy link
Contributor

@brokensound77 brokensound77 left a comment

Choose a reason for hiding this comment

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

The analysis makes sense to me.

Copy link
Contributor

@eric-forte-elastic eric-forte-elastic left a comment

Choose a reason for hiding this comment

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

Good discussion this morning. Based on that and what is written above, analysis makes good sense to me 👍

@terrancedejesus terrancedejesus merged commit b61da98 into main Jan 13, 2023
@terrancedejesus terrancedejesus deleted the google-workspace-min-stack-rules-8.4 branch January 13, 2023 18:29
protectionsmachine pushed a commit that referenced this pull request Jan 13, 2023
…2467)

* Bumping min-stack version for Google Workspace to 8.4

* changed 'updated_date' values

(cherry picked from commit b61da98)
protectionsmachine pushed a commit that referenced this pull request Jan 13, 2023
…2467)

* Bumping min-stack version for Google Workspace to 8.4

* changed 'updated_date' values

(cherry picked from commit b61da98)
protectionsmachine pushed a commit that referenced this pull request Jan 13, 2023
…2467)

* Bumping min-stack version for Google Workspace to 8.4

* changed 'updated_date' values

(cherry picked from commit b61da98)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants