kubernetes-sigs / azurelustre-csi-driver

82%

main: 87%

LAST BUILD ON BRANCH personal/dabradley/cleanupbadmounts

branch: personal/dabradley/cleanupbadmounts

CHANGE BRANCH

x

Reset

• personal/dabradley/cleanupbadmounts
• CVE-2022-32149
• CVE-2022-3821
• add-startup-taint
• broken
• bump-to-0.3.0
• change_ports
• chmuell/updateGo
• codeql-analysis
• codespell
• dependabot/go_modules/github.com/golang-jwt/jwt/v4-4.5.1
• dependabot/go_modules/github.com/golang-jwt/jwt/v4-4.5.2
• dependabot/go_modules/github.com/golang-jwt/jwt/v5-5.2.2
• dependabot/go_modules/golang.org/x/crypto-0.31.0
• dependabot/go_modules/golang.org/x/net-0.36.0
• dependabot/go_modules/golang.org/x/net-0.38.0
• dependabot/go_modules/k8s.io/kubernetes-1.31.6
• development
• disable_e2e
• disable_longhaul
• dummy
• dummy_branch
• dynamic-provisioning-preview
• dynamic-provisioning-to-development-merge
• feature/alwayspull-33532215
• feature/readiness-probe-29392102
• fix_badges
• fix_gated_checks
• fix_install
• fix_integration_test
• fixgatedcheckin
• fixlnet
• installpmcpkg
• iopsimpr
• ior3.3.0
• iortest
• link_fix_lnet
• longhaul
• longhaulImprove
• longhaulbug
• longhaulfix
• longhaulrun
• longhaulsecret
• longhual
• lustreversion
• main
• makemeowner
• mgsandothers
• mialve/dynamic-provisioning-preview
• mialve/fix-dalec-integration
• perf-scale-test
• personal/cjy/adddely
• personal/cjy/fixlnet
• personal/cjy/fixupdate
• personal/dabradley/1311kubeupgrades
• personal/dabradley/add-fs-group-policy
• personal/dabradley/changes-from-dynprov
• personal/dabradley/dynprov
• personal/dabradley/dynprov-release
• personal/dabradley/dynprov-update
• personal/dabradley/fix-tests
• personal/dabradley/fixtests
• personal/dabradley/lint
• personal/dabradley/trivy-on-development
• personal/dabradley/upgradecrypto
• personal/dabradley/upgradejwt
• personal/dabradley/upgradenet
• personal/dabradley/workflows
• personal/joatzinger-issue90a
• refs/tags/v0.1.17
• refs/tags/v0.1.18
• refs/tags/v0.2.0
• refs/tags/v0.3.0
• release0.1.1
• release0.1.2
• release0.1.3
• releasenote_0.1.4
• releasev0.1.0
• remoteinstall
• remove_space
• respond-to-comments
• revert-118-fix_install
• supportLustre2.15.1
• t-mialve/2.15.5-upgrade
• t-mialve/dalec-onboarding
• t-mialve/dalec-setup
• t-mialve/fix-dalec-integration
• t-mialve/fix-tests
• t-mialve/go-deps-update
• t-mialve/integration-testing
• t-mialve/skip-unknown-interfaces
• t-mialve/update-owners
• t-mialve/update-owners-development
• t-mialve/v0.1.16
• t-mialve/v0.1.17
• t-mialve/v0.1.18
• t-mialve/v0.2.0
• t-mialve/v0.3.0
• test_pr
• testbranch
• trivyerror
• ttrivy
• u2204
• update-kube-deps
• updateLonghaul
• updatecase
• upgrademod
• user/chyin/closemetricsserver
• user/chyin/copyrightinfo
• user/chyin/e2ecase
• user/chyin/loglevel
• user/chyin/mountoptiondoc
• user/chyin/mountoptions
• user/chyin/mountserial
• user/chyin/nomtab
• user/chyin/scaletest
• user/vinli/doc
• user/vinli/longhaul
• v0.1.18-m42
• v0.1.5
• v0.1.6doc
• v0.1.8
• v0.1.9
• waitcsiready

Build # 12018238717

Build Type

Pull #183

github

Committed by dabradley

Commit Message

Use force unmount and explicitly unmount bad mount points

There have been cases where the logic to cleanup a mount point
has caused the driver to get into a bad state. This is most
obvious when a subdirectory is mounted to a volume and a parent
directory of that subdirectory is deleted. The Lustre driver
doesn't handle that case in the way that Kubernetes expects
and returns invalid data. To avoid this scenario causing our
driver to get into a bad state, leak mount points, etc, we
must explicitly check that we can read the necessary information
about the mount point, and if not, explicitly unmount that
mount point before allowing Kubernetes to clean up the directory.
To ensure that we don't end up in a bad state, this change
enables force unmounting as well. The force unmount will only
occur after a timeout has expired, since force unmounts can
cause issues with the Lustre driver. However, in this case, it
is better if we are in a bad enough situation to be able to
eventually return to a good state rather than require manual
intervention.

Pull Request Pull Request #183: Use force unmount and explicitly unmount bad mount points

Run Details

18 of 41 new or added lines in 3 files covered. (43.9%)

1172 of 1432 relevant lines covered (81.84%)

6.96 hits per line