Remove non-Emissary executors

[alexec]

• Remove the deprecated docker executor #7795
• Remove k8sapi executor #7802
• Remove kubelet executor #7803
• Remove pns executor #7804

[terrytangyuan]

This would depend on resolving any limitations of the Emissary executor.

[alexec]

I think we can and should remove docker, k8sapi, and kublete executors. That would leave pns and emissary.

[terrytangyuan]

Sounds good. I can start with those.

[alexec]

I'd hold off for a few months. I don't think we want this until maybe v3.5.

[blkperl]

Wouldn't the removing part need to be done in a 4.0 release? We can add deprecation warnings in the logs and UI in the meantime though.

[alexec]

I actually think we can and should do this without going v4.0. The executor you use really should be an implementation detail. The emissary should have feature parity with other executors.

[terrytangyuan]

Just add a note here that we will also need to remove the functionality that allows using different executors.

[terrytangyuan]

Some learnings from @alexec's POC for future reference: #7930 (comment)

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[terrytangyuan]

Bump

[Brice187]

Is there a transion guide? All my workflows are affected

[terrytangyuan]

https://github.com/argoproj/argo-workflows/blob/master/docs/upgrading.md

[Brice187]

Okay, but how to fix the Workflows depending on another executor ;)

[terrytangyuan]

It's a breaking change. Follow the two items related to emissary in https://github.com/argoproj/argo-workflows/blob/master/docs/upgrading.md#upgrading-to-v34

[rossigee]

I'm having some issues with the default executor image, so I've git clone'd, added some debug lines to help me narrow down the problem and built a local container. How can I request that a workflow use my debug-enhanced container?