Both would need to be removed from kickstart template. There’s no guarantee people don’t use it in their custom templates where they set some value to these variables. I think we should either ask users first, this thread is tagged as Development. Or at least provide a deprecation warning or drawer notification if they are used before we remove them.
cc @ohadlevy since IIRC he was aware of someone using ks_console
while the macro is in the template, the code seems to indicate that these two both return empty strings since @grub, @port and @baud do not seem to be set anywhere. If we want to be on the safe side, we can add a deprecation warning to these and remove in 1.21.
I can’t remember anyone using it now atm, maybe we can somehow let users know about deprecation after an upgrade using the notification drawer? (or even scan if they have deprecated functions and then let them know…?)