-
Notifications
You must be signed in to change notification settings - Fork 6
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
Add deb and rpm package upgrade validation #94
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the approach makes sense!
Co-authored-by: Kevin Minehart <[email protected]>
Co-authored-by: Kevin Minehart <[email protected]>
Co-authored-by: Kevin Minehart <[email protected]>
pipelines/package_validate.go
Outdated
return fmt.Errorf("license in package does not match edition in package name") | ||
} | ||
} | ||
|
||
if taropts.Edition == "" { | ||
if err != nil || !strings.Contains(license, "GNU AFFERO GENERAL PUBLIC LICENSE") { | ||
return fmt.Errorf("failed to validate open-source license") | ||
if !strings.Contains(license, "GNU AFFERO GENERAL PUBLIC LICENSE") { | ||
return fmt.Errorf("license in package does not match edition in package name") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
in these contexts we know what to expect; we should be clearer about what we expect vs what was given. "license in package is not the Grafana Enterprise license agreement", for example.
No description provided.