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

Next Version 2.4.10 #2206

Merged
merged 1 commit into from
Jan 15, 2025
Merged

Next Version 2.4.10 #2206

merged 1 commit into from
Jan 15, 2025

Conversation

fax4ever
Copy link
Collaborator

@fax4ever fax4ever commented Jan 13, 2025

@fax4ever fax4ever marked this pull request as draft January 13, 2025 10:35
@fax4ever
Copy link
Collaborator Author

I made the PR too early. The errors are related to the fact that the bundle is not already present:

https://quay.io/repository/operatorhubio/infinispan?tab=tags&tag=v2.4.9

I re-run the CI as soon as we have it.

@fax4ever fax4ever marked this pull request as ready for review January 13, 2025 12:17
@@ -17,6 +17,7 @@ func LogError(err error) {
// ExpectNoError checks if an error exists, and if so halts execution
func ExpectNoError(err error) {
if err != nil {
LogError(err)
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this is redundant as we already log the error in kubernetes.go#CleanNamespaceAndLogWithPanic after the panic is thrown.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

You're right. In fact, I have the same results. I'm removing it. Thanks

@ryanemerson ryanemerson merged commit 817a8a5 into infinispan:main Jan 15, 2025
19 of 21 checks passed
@ryanemerson
Copy link
Contributor

Thanks @fax4ever

@fax4ever fax4ever deleted the update-2.4.10 branch January 15, 2025 18:11
@fax4ever
Copy link
Collaborator Author

Thanks

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.

2 participants