omgx integration

[InoMurko]

No description provided.

[InoMurko]

I've deleted omgx-sync-tests.yml because I think it's a duplicate of sync-tests.yml.

These tests really just test verifier and replica.

omgx-integration.yml has commented out the original Optimism integration tests. The old develop had them enabled and the reason for it was that it allowed us check whether our containers somehow interfere with the original containers. Just putting this out there in case @jliphard decides to enable them back.

[InoMurko]

I see that creating an omgx docker - compose overlay was commented out:

# yq eval '(.services.[].image | select(. == "ethereumoptimism*")) |= sub("ethereumoptimism", "omgx")' ${ORIGINAL_DOCKERFILE} | \
# yq eval '(.services.[].image) += ":latest"' - \
# > ${DOCKERFILE}

This means we're relying on Optimisms push of container images into their docker hub. Not ours. So their latest container could in in front of our containers and there could be discrepancies the way the system is functioning.

If you use the up_local pull functionality without omgx docker compose overlay, you get very old containers:

REPOSITORY                              TAG       IMAGE ID       CREATED        SIZE
omgx/omgx_deployer                      latest    3b1a522e7408   8 hours ago    1.01GB
omgx/omgx_builder                       latest    19d815f22f6e   8 hours ago    3.76GB
ethereumoptimism/builder                latest    1f952ca4ae99   5 days ago     1.14GB
ethereumoptimism/deployer               latest    538e0b70d62b   **2 months ago**   1.54GB
ethereumoptimism/batch-submitter        latest    d525dc90394e   **2 months ago**   1.53GB
ethereumoptimism/data-transport-layer   latest    f9c6ad8f02ed   **2 months ago**   1.39GB
ethereumoptimism/message-relayer        latest    0b1e3a1fe302   **2 months ago**   1.25GB
ethereumoptimism/integration-tests      latest    3ceed3144a81   **2 months ago**   2.22GB
ethereumoptimism/hardhat                latest    3a2236160798   **3 months ago**   1.12GB

[codecov-commenter]

Codecov Report

Merging #146 (f965e0e) into develop (c7ebd2c) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #146   +/-   ##
========================================
  Coverage    86.05%   86.05%           
========================================
  Files           49       49           
  Lines         1936     1936           
  Branches       307      307           
========================================
  Hits          1666     1666           
  Misses         270      270           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c7ebd2c...f965e0e. Read the comment docs.

[CAPtheorem]

Most changes incorporated into #149.

[InoMurko]

okay, so should this be closed?

[CAPtheorem]

yes, can be closed