racetrack icon indicating copy to clipboard operation
racetrack copied to clipboard

If there are multiple kubernetes infrastructure target plugins installed, a single job are listed in multiple infrastructure targets in the admin page.

Open anders314159 opened this issue 1 year ago • 2 comments

image

I think it's kinda confusing that hovering the blue 1 in the screenshot says "1 Jobs is using this infrastructure target" for both 1.5.1 and 1.5.0. Do jobs really run on two IF targets at the same time?

anders314159 avatar Jul 03 '24 12:07 anders314159

Right, that might be confusing and misleading. I think racetrack should prohibit that kind of situation innately. It doesn't make sense to have multiple versions of the same infrastructure targets. If they coexist, they're fighting with each other. They discover the same jobs twice and they step on each other's toes by overwriting the same jobs. Fortunately, when deploying a job, only the one (newer) is used.

At this point, plugins can be installed multiple times with different versions. That makes sense for job type plugins, whereas it's kind of pointless for infra plugins. So maybe we could restrict this behavior when detecting "infrastructure" plugin.

iszulcdeepsense avatar Jul 08 '24 10:07 iszulcdeepsense

If there already is an infrastructure plugin with the same name, we can replace it. Should we require that the replace flag is set before replacing the infrastructure plugin, or should it just be default that the user replaces the previous infrastructure plugin when they upload a new one?

anders314159 avatar Aug 16 '24 09:08 anders314159

If there already is an infrastructure plugin with the same name, we can replace it. Should we require that the replace flag is set before replacing the infrastructure plugin, or should it just be default that the user replaces the previous infrastructure plugin when they upload a new one?

Good question. I don't have a strong opinion here. I think Racetrack could raise an error and require replacing it explicitly.

iszulcdeepsense avatar Aug 19 '24 11:08 iszulcdeepsense