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

[FR] Back-porting Version Trimming #3563

Closed
shashank-elastic opened this issue Apr 2, 2024 · 5 comments · Fixed by #3681
Closed

[FR] Back-porting Version Trimming #3563

shashank-elastic opened this issue Apr 2, 2024 · 5 comments · Fixed by #3681
Assignees
Labels
Area: DED enhancement New feature or request

Comments

@shashank-elastic
Copy link
Contributor

Is your feature request related to a problem? Please describe.
Keep a check on the number of versions that the updates are back ported to. Thumb Rule is to stick with. release cadence of latest and latest-3 versions

Describe the solution you'd like
Limit back porting to 8.10 as of now we are running GA of 8.13

Describe alternatives you've considered
NA

@shashank-elastic shashank-elastic added enhancement New feature or request Area: DED labels Apr 2, 2024
@shashank-elastic shashank-elastic self-assigned this Apr 2, 2024
@shashank-elastic
Copy link
Contributor Author

Essentially:

  • Dropping validation checks to latest - 3 (stack-schema-map.yaml)
  • Backport checks will only be latest - 3 as well (lock-versions.yml)
  • Moving forward we will always drop one then bumping to the next version.
  • We need to decide when to make this change.

@Mikaayenson
Copy link
Collaborator

Mikaayenson commented Apr 2, 2024

  • Prior to trimming during the release, tag the branch.
  • We should release - 3, but preserve - 5
  • Revise test_previous_entries_gte_current_min_stack

@shashank-elastic
Copy link
Contributor Author

This has to be completed this sprint cycle, Making this Urgent from conversation with @Mikaayenson

@shashank-elastic
Copy link
Contributor Author

  • Discussion On test_previous_entries_gte_current_min_stack

    • Should we really trim lock versions, if we find no value on this then we really dont need to run a trim.
    • IF we decide no trimming on lock versions file is needed this test case has to be removed.
    • If we don't remove this test case, then we always have to trim every minor release and update min_stack to rules across.
  • We are clear on trimming to 5 versions and releasing to 3

  • It was decided to open a separate PR with necessary changes and put this in Design Review for further discussions.

  • Please note this has to be completed in this cycle, so will be driving this 🔥

@shashank-elastic
Copy link
Contributor Author

PR Up for review : #3681

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: DED enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants