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

Fix changelog in network traffic #11322

Merged
merged 2 commits into from
Oct 14, 2024

Conversation

jsoriano
Copy link
Member

@jsoriano jsoriano commented Oct 3, 2024

1.31.1 and 1.31.2 were released after 1.32.0, including more changes than 1.32.0. Upgrading to 1.32.0 actually causes regression as there is one change that is not included.

Adjust the changelog to represent the current state of packages, and release a new 1.32.1 version that includes the latest changes.

We detected this issue in elastic/package-spec#808, adding validations to try to prevent this kind of problems.

@jsoriano jsoriano self-assigned this Oct 3, 2024
@jsoriano jsoriano requested a review from a team as a code owner October 3, 2024 12:25
@andrewkroh andrewkroh added Integration:network_traffic Network Packet Capture Team:Security-Linux Platform Linux Platform Security team [elastic/sec-linux-platform] labels Oct 3, 2024
@elasticmachine
Copy link

Pinging @elastic/sec-linux-platform (Team:Security-Linux Platform)

@elastic-vault-github-plugin-prod
Copy link

elastic-vault-github-plugin-prod bot commented Oct 3, 2024

🚀 Benchmarks report

Package network_traffic 👍(0) 💚(0) 💔(1)

Expand to view
Data stream Previous EPS New EPS Diff (%) Result
memcached 1834.86 832.64 -1002.22 (-54.62%) 💔

To see the full report comment with /test benchmark fullreport

@jsoriano
Copy link
Member Author

@andrewkroh @efd6 can I get a review on this? Another change was merged on the incorrect version.

In latest elastic-package we are adding a check to prevent this, network traffic is failing because of this in #11400.

@elasticmachine
Copy link

💚 Build Succeeded

History

cc @jsoriano

@jsoriano
Copy link
Member Author

Thanks!

@jsoriano jsoriano merged commit 45d2434 into elastic:main Oct 14, 2024
5 checks passed
@jsoriano jsoriano deleted the network-traffic-changelog branch October 14, 2024 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Integration:network_traffic Network Packet Capture Team:Security-Linux Platform Linux Platform Security team [elastic/sec-linux-platform]
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants