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

[feat] add Ubuntu 24.04 Noble package support #971

Merged
merged 1 commit into from
Nov 22, 2024

Conversation

flyinprogrammer
Copy link
Contributor

In theory this might capture this request for an issue! 🌮

Copy link

codecov bot commented Aug 31, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.44%. Comparing base (fea49bc) to head (ebd109e).
Report is 179 commits behind head on unstable.

Additional details and impacted files
@@             Coverage Diff              @@
##           unstable     #971      +/-   ##
============================================
- Coverage     70.50%   70.44%   -0.06%     
============================================
  Files           114      114              
  Lines         61742    61742              
============================================
- Hits          43532    43495      -37     
- Misses        18210    18247      +37     

see 12 files with indirect coverage changes

---- 🚨 Try these New Features:

Copy link
Member

@roshkhatri roshkhatri left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@melroy89 melroy89 left a comment

Choose a reason for hiding this comment

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

Not sure why the CICD test fails. But this change LGTM. Thx!

@madolson
Copy link
Member

madolson commented Oct 6, 2024

Not sure why the CICD test fails. But this change LGTM. Thx!

We had some instability around when we launched Valkey 8 because of the features. You can find most of the documented issues here: https://github.com/valkey-io/valkey/issues?q=is%3Aopen+is%3Aissue+label%3Atest-failure. The CI pretty stable generally.

@@ -12,6 +12,12 @@
"type": "deb",
"platform": "focal"
},
{
"arch": "x86_64",
"target": "ubuntu24.04",
Copy link
Member

Choose a reason for hiding this comment

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

@roshkhatri I see we are skipping 22.04. I'm Okay adding these, but maybe we should document our strategy here somewhere?

Copy link
Contributor

Choose a reason for hiding this comment

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

Like focus on Ubuntu LTS releases would make sense to me.

Copy link
Member

Choose a reason for hiding this comment

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

@roshkhatri @bbarani Can we make an issue to document our strategy and follow through on updating the website. I think I'll merge this now and we can follow up on the next release adding these images.

Copy link
Member

Choose a reason for hiding this comment

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

Yes I have opened an issue for the website repo here valkey-io/valkey-io.github.io#164.

Copy link
Member

Choose a reason for hiding this comment

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

Also I think there is a problem, for compiling, we use https://github.com/marketplace/actions/run-on-architecture for the binaries for ARM architecture. Github ARM runners are in GA but as github-hosted runners, there are not in opensource according to https://github.com/orgs/community/discussions/19197#discussioncomment-10550689.

I will look into more options.

@madolson madolson merged commit 377ed22 into valkey-io:unstable Nov 22, 2024
45 of 46 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants