Skip to content

Fixes the and removes other networks.#200

Merged
lasarojc merged 1 commit intomainfrom
lasarojc/199/clean-up-network-folder
Jan 25, 2023
Merged

Fixes the and removes other networks.#200
lasarojc merged 1 commit intomainfrom
lasarojc/199/clean-up-network-folder

Conversation

@lasarojc
Copy link
Contributor

@lasarojc lasarojc commented Jan 25, 2023

Fixes the networks/local/localnode/wrapper.sh and removes other networks and corresponding documentation.


PR checklist

  • Tests written/updated, or no tests needed
  • CHANGELOG_PENDING.md updated, or no changelog entry needed
  • Updated relevant documentation (docs/) and code comments, or no
    documentation updates needed

@lasarojc lasarojc linked an issue Jan 25, 2023 that may be closed by this pull request
@lasarojc lasarojc self-assigned this Jan 25, 2023
@lasarojc lasarojc marked this pull request as ready for review January 25, 2023 18:14
@lasarojc lasarojc requested a review from a team as a code owner January 25, 2023 18:14
Copy link
Contributor

@thanethomson thanethomson left a comment

Choose a reason for hiding this comment

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

It's really great seeing so many unmaintained/legacy lines of code/config disappearing 🙂 It already reduces the cognitive burden of maintaining this particular codebase.

@lasarojc lasarojc merged commit 5d54939 into main Jan 25, 2023
@lasarojc lasarojc deleted the lasarojc/199/clean-up-network-folder branch January 25, 2023 19:41
mergify bot pushed a commit that referenced this pull request Jan 25, 2023
(cherry picked from commit 5d54939)

# Conflicts:
#	docs/networks/terraform-and-ansible.md
#	networks/remote/README.md
#	networks/remote/ansible/roles/logzio/tasks/main.yml
mergify bot pushed a commit that referenced this pull request Jan 25, 2023
(cherry picked from commit 5d54939)

# Conflicts:
#	docs/networks/terraform-and-ansible.md
#	networks/remote/README.md
#	networks/remote/ansible/config.yml
#	networks/remote/ansible/logzio.yml
#	networks/remote/ansible/roles/logzio/tasks/main.yml
lasarojc added a commit that referenced this pull request Jan 26, 2023
* Fixes the  and removes other networks. (#200)

(cherry picked from commit 5d54939)

# Conflicts:
#	docs/networks/terraform-and-ansible.md
#	networks/remote/README.md
#	networks/remote/ansible/roles/logzio/tasks/main.yml

* Resolving backport conflicts

Co-authored-by: Lasaro <lasaro@informal.systems>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
lasarojc added a commit that referenced this pull request Jan 26, 2023
* Fixes the  and removes other networks. (#200)

(cherry picked from commit 5d54939)

# Conflicts:
#	docs/networks/terraform-and-ansible.md
#	networks/remote/README.md
#	networks/remote/ansible/config.yml
#	networks/remote/ansible/logzio.yml
#	networks/remote/ansible/roles/logzio/tasks/main.yml

* Resolve mergify backport conflicts

Co-authored-by: Lasaro <lasaro@informal.systems>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rename Renaming our fork

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

Clean up and do renaming in the network

2 participants