Skip to content

Add new varnish versions #253

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

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

Conversation

fozboz
Copy link

@fozboz fozboz commented Mar 24, 2025

Added all 7.x versions to the list of allowed versions.

Made the default 7.7.

Description

Supports newer varnish versions (there are CVEs on < 7.6.2)

Added all 7.x versions to the list of allowed versions.

Made the default 7.7.
@fozboz fozboz requested a review from a team as a code owner March 24, 2025 20:07
@Stromweld Stromweld added Release: Minor Release to Chef Supermarket as a minor release when merged Release: Major Release to Chef Supermarket as a major change when merged and removed Release: Minor Release to Chef Supermarket as a minor release when merged labels Mar 24, 2025
@Stromweld
Copy link

Please undo the metadata version change as our pipeline automates that. Can you also fix the markdown issues and remove the broken links.

Not sure why we'd want to check version and for specific versions but that would be a recommendation to remove for a future refactor of this CB to be more future compatible if nothing changes to install process.

@fozboz
Copy link
Author

fozboz commented Mar 25, 2025

Done.

I agree, looks like that was needed prior due to version differences. For now, I just need to get varnish upgraded.

@Stromweld
Copy link

@fozboz do you mind if I clone and push an update to your fork for this PR. Trying to look at fixing the ubuntu and debian integration tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Release: Major Release to Chef Supermarket as a major change when merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants