Support tags with colon (:) about dagster HOT 4 OPEN

Relates to this commit: b326b97

from dagster.

Hey @zyd14 - this isn't set in stone, but the two main reasons we went in this direction were:

• Tags are key-value pairs, and it's easy for "a:b" to look like "a" is the key and "b" is the value
• We largely cargo-culted the Kubernetes syntax for labels, for the sake of consistency and because they seem to have a pretty principled approach

from dagster.

Fair points. We took inspiration from AWS as that's where most of our tags surface anyway and they do colon-separated namespaces for keys. I've ran into a few services that don't allow it now though which makes me feel like maybe we've got it wrong.

from dagster.

To be clear, I think it is extremely unlikely that we'd actually be able to drop support for these keys. The deprecation warning is there to guide new users towards a particular direction, which will allow us to build UI that favors it (e.g. visually distinguishing the part before and after the "/").

If the deprecation warning itself is cramping your style, we might be able to add a way to disable it more comprehensively in your setup?

from dagster.