Commit 7d2bde1 (Dec 8 2017) adds the TODO item, and excludes
the "should.support.inline.execution.and.attach" test case.

Now, commit 54d1971 (Jun 24 2019) removes this test from the
exclude regexp, but forgots to remove the TODO.

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>

In its current form, it is very hard to read, maintain, or make any
sense out of.

Play with ansible/jinja a bit to enable much simpler way to list
the tests to skip.

The downside is we have to play some tricks to create a regular
expression:

 1. join the elements of skip_tests via |
 2. replace spaces with \\s
 3. prepend square brackets and parentheses with \\

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>

See the previous commit for details.

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>

This is mostly done to fix the flake caused by the test case

> Kubernetes e2e suite: [sig-network] IngressClass [Feature:Ingress] should not set default value if no default IngressClass [Serial]

which fails like this:

> ingressclasses.networking.k8s.io "ingressclass1" already exists

This test depends on a system state and thus can't be run in parallel.
It was marked as [Serial] recently (see [1]) but I just found out
we do not skip Serial tests in e2e-features.

Fix this.

[1] kubernetes/kubernetes#93427

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>