Skip to content

Commit

Permalink
Merge pull request #2980 from gj199575/deleteCMResourceLock
Browse files Browse the repository at this point in the history
when Volcano is uninstalled, two resources will remain
  • Loading branch information
volcano-sh-bot authored Jul 20, 2023
2 parents f9d22d6 + 19bc0f3 commit 53cce5d
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion cmd/controller-manager/app/server.go
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,7 @@ func Run(opt *options.ServerOption) error {
// add a uniquifier so that two processes on the same host don't accidentally both become active
id := hostname + "_" + string(uuid.NewUUID())

rl, err := resourcelock.New(resourcelock.ConfigMapsLeasesResourceLock,
rl, err := resourcelock.New(resourcelock.LeasesResourceLock,
opt.LockObjectNamespace,
"vc-controller-manager",
leaderElectionClient.CoreV1(),
Expand Down
2 changes: 1 addition & 1 deletion cmd/scheduler/app/server.go
Original file line number Diff line number Diff line change
Expand Up @@ -125,7 +125,7 @@ func Run(opt *options.ServerOption) error {
// add a uniquifier so that two processes on the same host don't accidentally both become active
id := hostname + "_" + string(uuid.NewUUID())

rl, err := resourcelock.New(resourcelock.ConfigMapsLeasesResourceLock,
rl, err := resourcelock.New(resourcelock.LeasesResourceLock,
opt.LockObjectNamespace,
commonutil.GenerateComponentName(opt.SchedulerNames),
leaderElectionClient.CoreV1(),
Expand Down

0 comments on commit 53cce5d

Please sign in to comment.