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

Update Falco bumping instructions #1807

Merged
merged 1 commit into from
Aug 22, 2024

Conversation

Molter73
Copy link
Collaborator

@Molter73 Molter73 commented Aug 21, 2024

Description

With the removal of the MODULE_VERSION file in #1773 our handling of the falco libs fork is somewhat outdated, since now there is no module version to use as a guideline. Instead, I'm proposing that we start using the upstream version we are based on as part of our branching strategy.

With this new approach, branches with our changes will be of the form M.m.p-stackrox, with M.m.p being the version numbers for the upstream libs tag the branch is based on.

Checklist

  • Investigated and inspected CI test results
  • Updated documentation accordingly

Automated testing

  • Added unit tests
  • Added integration tests
  • Added regression tests

If any of these don't apply, please comment below.

Testing Performed

This is a documentation only PR, no testing is necessary.

With the removal of the MODULE_VERSION file in #1773 our handling of
the falco libs fork is somewhat outdated, since now there is no module
version to use as a guideline. Instead, I'm proposing that we start
using the upstream version we are based on as part of our branching
strategy.

With this new approach, branches with our changes will be of the form
M.m.p-stackrox, with M.m.p being the version numbers for the upstream
libs tag the branch is based on.
@Molter73 Molter73 requested a review from a team as a code owner August 21, 2024 13:24
Copy link
Contributor

@ovalenti ovalenti left a comment

Choose a reason for hiding this comment

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

I think that this new scheme makes sense. Although @Stringy should have the final word here.

Copy link
Collaborator

@Stringy Stringy left a comment

Choose a reason for hiding this comment

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

LGTM! everything will be a lot clearer now 👌

@Molter73 Molter73 merged commit 0eea6ea into master Aug 22, 2024
52 of 62 checks passed
@Molter73 Molter73 deleted the mauro/update-falco-bumping-instructions branch August 22, 2024 09:28
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