Skip to content
This repository has been archived by the owner on Oct 31, 2024. It is now read-only.

chore: cleanup topos tools #397

Merged
merged 7 commits into from
Dec 20, 2023
Merged

chore: cleanup topos tools #397

merged 7 commits into from
Dec 20, 2023

Conversation

atanmarko
Copy link
Member

Description

Remove topos docker-compose setup and related configurations and assets

Fixes TP-807

docker-compose push-certificate check test will be refactored in additional PR to run as rust standalone test.
Additional tests will be developed on local-erc20-messaging-infra.

PR Checklist:

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added or updated tests that comprehensively prove my change is effective or that my feature works

@atanmarko atanmarko self-assigned this Dec 6, 2023
@atanmarko atanmarko marked this pull request as ready for review December 6, 2023 12:14
@atanmarko atanmarko requested a review from a team as a code owner December 6, 2023 12:14
README.md Outdated Show resolved Hide resolved
grafana/provisioning/dashboards/cadvisor.json Outdated Show resolved Hide resolved
grafana/provisioning/dashboards/dashboard.yml Outdated Show resolved Hide resolved
Copy link
Member

@Freyskeyd Freyskeyd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR needs to be merge after we have running e2e tests on local-erc20

Dockerfile Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@CLAassistant
Copy link

CLAassistant commented Dec 20, 2023

CLA assistant check
All committers have signed the CLA.

@atanmarko atanmarko merged commit 0820306 into main Dec 20, 2023
17 checks passed
@atanmarko atanmarko deleted the chore/TP-807 branch December 20, 2023 10:56
@@ -45,10 +45,25 @@ topos --help

Find more about how topos works in the [documentation](https://docs.topos.technology/).

### Topos Docker image
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure why this sub-section in the main README btw. This somewhat differs from the rest of the README that is mainly for node runners, otherwise we do have docker instruction in docs.topos.technology.

cc @wyhaines

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should I remove it?

Run Topos docker image:

```
docker run -it --rm topos:latest --help
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

--help?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants