Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

fix: isContainerdEnabled checks driver status field instead of driver #19

Merged
merged 1 commit into from
Jul 19, 2024
Merged

fix: isContainerdEnabled checks driver status field instead of driver #19

merged 1 commit into from
Jul 19, 2024

Conversation

jgrer
Copy link
Collaborator

@jgrer jgrer commented Jul 19, 2024

  • Changed the isContainerdEnabled function to check the driver status field for "driver-type io.containerd.snapshotter.v1" instead of checking the Driver field for "overlay2" or "overlayfs"

@jgrer
fix: changed isContainerdEnabled to check the driver status for conta…
6252a7e 
…inerd instead of the Driver field

Signed-off-by: Jackson Greer <t-jgreer@microsoft.com>
ashnamehrotra
ashnamehrotra approved these changes Jul 19, 2024
View reviewed changes
Copy link
Contributor

@ashnamehrotra ashnamehrotra left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@ashnamehrotra ashnamehrotra merged commit 74fff4c into project-copacetic:main Jul 19, 2024
2 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@ashnamehrotra ashnamehrotra ashnamehrotra approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jgrer @ashnamehrotra