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

start development towards 7.0.4 #10342

Closed
wants to merge 1 commit into from

Conversation

inashivb
Copy link
Member

@inashivb inashivb commented Feb 9, 2024

No description provided.

@inashivb inashivb requested a review from a team as a code owner February 9, 2024 04:33
Copy link

codecov bot commented Feb 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (be68bbc) 82.25% compared to head (6b0ff3e) 82.25%.

Additional details and impacted files
@@              Coverage Diff               @@
##           main-7.0.x   #10342      +/-   ##
==============================================
- Coverage       82.25%   82.25%   -0.01%     
==============================================
  Files             975      975              
  Lines          274915   274915              
==============================================
- Hits           226128   226125       -3     
- Misses          48787    48790       +3     
Flag Coverage Δ
fuzzcorpus 63.51% <ø> (+0.02%) ⬆️
suricata-verify 61.18% <ø> (-0.01%) ⬇️
unittests 62.88% <ø> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

libhtp https://github.com/OISF/libhtp 0.5.46
suricata-update https://github.com/OISF/suricata-update 1.3.0
libhtp https://github.com/OISF/libhtp 0.5.x
suricata-update https://github.com/OISF/suricata-update master
Copy link
Member

Choose a reason for hiding this comment

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

why set this to master? It was 1.3.0 before the release

Copy link
Member Author

Choose a reason for hiding this comment

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

I think this is what Jason and I agreed on. But, I'll let him answer in case I misunderstood.

  • The thought is that just like the latest version of libhtp goes w a dev branch of suricata, latest version of a respective suricata-update branch should go as well.
  • Given that we don't have a 1.3.x branch, current master of suricata-update represents the latest state that we use and bundle with suricata's 7.0.x.
  • Similarly, for suricata 6.0.x, we use the branch 1.2.x of suricata-update as that reflects the latest state of what should be used and bundled.
  • Please note that for now, in suricata-update,
    branch master == tag 1.3.0 and,
    branch master-1.2.x == tag 1.2.8

Copy link
Member

Choose a reason for hiding this comment

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

Ok, thanks for clearing that up.

Copy link
Member

Choose a reason for hiding this comment

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

I missed this, only noticed because I was looking for the commit ID for a forward port :)

This works as long as we pin the versions and commit before we tag Suricata.

So will the dev tools take care of this? For example, when I go and checkout the tag suricata-7.0.4 (after its released of course), I should see specific versions in this file.

Copy link
Member Author

Choose a reason for hiding this comment

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

yes, we already do that :)
Prep stage: the requirements.txt gets an update of specific libhtp and suricata-update version that needs to be bundled in the release. e.g. be68bbc#diff-4d7c51b1efe9043e44439a949dfd92e5827321b34082903477fd04876edb7552
Post release stage: the requirements.txt go back to the libhtp and suricata-update branches corresponding to the respective suricata version. e.g. this PR

@inashivb inashivb requested a review from jasonish February 9, 2024 08:57
@victorjulien
Copy link
Member

Can you combine this with CI backports just merged in master?

@inashivb inashivb mentioned this pull request Feb 9, 2024
@inashivb inashivb closed this Feb 9, 2024
@inashivb inashivb deleted the dev-7.0.4/v1 branch February 9, 2024 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants