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

NAS-131449 / 24.10-RC.1 / fix bug in deprecated service detection (by yocalebo) #734

Merged
merged 1 commit into from
Sep 30, 2024

Conversation

bugclerk
Copy link

I'm not even really sure how this has worked in the first place....QE has a system (running 24.04.2.2) with a Gold license that they're manually upgrading to what will be EE-RC.1. They're met with a deprecation message and the update fails. After investigation, I'm not sure how this has ever worked....

The system they had was running minio which produced a running service name of minio but the crgroup entry was

0::/kubepods.slice/kubepods-burstable.slice/kubepods-burstable pod3c3941c1_24e7_4189_854f_f9dfcfa638ef.slice/cri-containerd-14ef8d9c8a76f8c8f1731bd455062481c4e03f683fe52566bbc4fc74f480225.scope

I went ahead and installed webdav app and also saw it produce an entry like so

0::/kubepods.slice/kubepods-burstable.slice/kubepods-burstable-podf905dab1_0a2c_4e68_9d0c_861da1874869.slice/cri-containerd-660a2b0eb91bc45f92b245099be5b2d27686d2e29329bc2eeeedc8f4a0b951e0.scope

This means we're tripping the deprecation service logic rather easily and after looking at it for a bit, I'm not sure how we were ever doing this properly 🤔 Anyways, I've added the kubepod string as another check to prevent these false positive alerts.

Original PR: #733
Jira URL: https://ixsystems.atlassian.net/browse/NAS-131449

(cherry picked from commit efc5969)
@yocalebo yocalebo merged commit 14a4ee2 into release/24.10-RC.1 Sep 30, 2024
3 of 4 checks passed
@yocalebo yocalebo deleted the NAS-131449-24.10-RC.1 branch September 30, 2024 18:58
@bugclerk
Copy link
Author

This PR has been merged and conversations have been locked.
If you would like to discuss more about this issue please use our forums or raise a Jira ticket.

@truenas truenas locked as resolved and limited conversation to collaborators Sep 30, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants