volcano icon indicating copy to clipboard operation
volcano copied to clipboard

Can not sync job status correctly when upgrading from v1.5

Open yuyue9284 opened this issue 1 year ago • 4 comments

What happened:

When upgrading volcano from the v1.5 to later version, the already running job created by the v1.5 volcano cannot be correctly handled.

Anything else we need to know?:

v1.5 changed the naming logics of pod group by adding UID into the name: #2140, and there is also another fix regarding handling the already created pod group without UID in create or update: #2400. But a similar fix does not exist in the syncJob function.

So, in the following part the syncTask won't be set to true.

https://github.com/volcano-sh/volcano/blob/1a696ee33c88a9b3f1bf3c43f272aacfc35121be/pkg/controllers/job/job_controller_actions.go#L280-L293

Environment:

  • Volcano Version:
  • Kubernetes version (use kubectl version):
  • Cloud provider or hardware configuration:
  • OS (e.g. from /etc/os-release):
  • Kernel (e.g. uname -a):
  • Install tools:
  • Others:

yuyue9284 avatar Jul 30 '24 03:07 yuyue9284

Yeah you're right, we haven't consider the old pg format when syncjob: )

Monokaix avatar Jul 30 '24 06:07 Monokaix

/good-first-issue

Monokaix avatar Jul 30 '24 06:07 Monokaix

@Monokaix: This request has been marked as suitable for new contributors.

Please ensure the request meets the requirements listed here.

If this request no longer meets these requirements, the label can be removed by commenting with the /remove-good-first-issue command.

In response to this:

/good-first-issue

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

volcano-sh-bot avatar Jul 30 '24 06:07 volcano-sh-bot

/assign

QingyaFan avatar Aug 02 '24 14:08 QingyaFan

fixed by https://github.com/volcano-sh/volcano/pull/3786

Monokaix avatar Oct 22 '24 06:10 Monokaix

/close

Monokaix avatar Oct 22 '24 06:10 Monokaix

@Monokaix: Closing this issue.

In response to this:

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

volcano-sh-bot avatar Oct 22 '24 06:10 volcano-sh-bot