Reviewed 12 of 12 files at r1.
Reviewable status: all files reviewed, 6 unresolved discussions (waiting on @lukedirtwalker)

scion.sh, line 56 at r1 (raw file):

        if is_docker_be; then
            echo "Build perapp images"
            ./tools/quiet make -C docker/perapp bazel

I would drop the quite script.
Personally I would prefer to avoid having another layer of wrapper.
bazel run //docker/perapp:prod should be enough

scion.sh, line 58 at r1 (raw file):

            ./tools/quiet make -C docker/perapp bazel
            echo "Build scion tester"
            ./tools/quiet ./docker.sh tester

bazel run //docker/testimages:scion_testing_bundle 

.buildkite/pipeline.yml, line 98 at r1 (raw file):

  - label: ":docker: Integration: end2end_integration"
    command:
    - ./scion.sh topology -t -d -c topology/Tiny.topo

I would have bazel build the images in bullet before the run step.

Tiny or Tiny4?

.buildkite/pipeline.yml, line 100 at r1 (raw file):

    - ./scion.sh topology -t -d -c topology/Tiny.topo
    - ./scion.sh run
    - ./tools/dc start 'tester*'

dc is a wrapper to docker-compose, it's worth start reducing the usage of it

topology/Tiny.topo, line 1 at r1 (raw file):

--- # Tiny Topology

Tiny6 should be better name now

topology/Tiny4.topo, line 1 at r1 (raw file):

--- # Tiny Topology IPv4 only

why we need to introduce a Tiny with v4?

Reviewable status: 11 of 12 files reviewed, 6 unresolved discussions (waiting on @karampok)

scion.sh, line 56 at r1 (raw file):

this wouldn't build the SIG container, which might be needed for some users.

scion.sh, line 58 at r1 (raw file):

this doesn't build the sig testing image, which might be needed for some users.

.buildkite/pipeline.yml, line 98 at r1 (raw file):

Tiny

.buildkite/pipeline.yml, line 100 at r1 (raw file):

done.

topology/Tiny.topo, line 1 at r1 (raw file):

No, I think this should be the new default. v4 should be deleted.

topology/Tiny4.topo, line 1 at r1 (raw file):

because some acceptance tests are built with v4 in mind. I don't want to change them right now.

Reviewed 1 of 1 files at r2.
Reviewable status: complete! all files reviewed, all discussions resolved

Reviewable status: complete! all files reviewed, all discussions resolved

Reviewed 1 of 1 files at r3.
Reviewable status: complete! all files reviewed, all discussions resolved