Skip to content

ci: deploy operator with ansible - #1382

Merged
JuArce merged 46 commits into
stagingfrom
operator_ansible
Nov 13, 2024
Merged

ci: deploy operator with ansible#1382
JuArce merged 46 commits into
stagingfrom
operator_ansible

Conversation

@JuArce

@JuArceJuArce commented Nov 5, 2024

Copy link
Copy Markdown
Collaborator

Ansible setup for operator

Note

Merge after #1384

Description

This PR adds the setup for the operator

The current inventory is set to deploy the operator to aligned-holesky-stage-1 server.

  1. First create the env files:
make ansible_operator_create_env
  1. Then complete the env files.

  2. Deploy the operator

make ansible_operator_deploy INVENTORY=./infra/ansible/stage_inventory.yaml ECDSA_KEYSTORE=<> BLS_KEYSTORE=<>

How to Test

Follow the README instructions

Type of change

  • CI

@JuArceJuArce self-assigned this Nov 5, 2024
Base automatically changed from batcher_ansible to 1344-ci-improve-servers-setupNovember 5, 2024 21:04
@JuArce
JuArce marked this pull request as draft November 6, 2024 13:21
@JuArce
JuArce marked this pull request as ready for review November 7, 2024 16:12
- name: Run setup playbook
ansible.builtin.import_playbook: setup.yaml
vars:
host: operator

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

For a future improvement: parametrize this variable so it's not repeated so much

Comment threadMakefile

@MarcosNicolauMarcosNicolau left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Small nit comments. Everything worked alright. I believe a small comment for each of the fields in the config would be great. I had a hard time setting them up because I didn't understand where they should point to.

Comment threadinfra/ansible/README.md
Comment threadinfra/ansible/playbooks/ini/config-operator.ini.example
Comment threadinfra/ansible/playbooks/operator.yaml Outdated
MarcosNicolau
MarcosNicolau approved these changes Nov 12, 2024

@klaus993klaus993 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

It's really important that in the short term we also parametrize the playbook to send it the stage which we want to deploy (stage, testnet or mainnet) so default values can be set automatically.
Right now the ini files need to be populated manually and many of the values can be easily filled in automatically according to the stage (the values shown in the table of the README).
We need to have as minimal setup as possible so it's easier if we're in a hurry, and these values I'm talking about are stuff that doesn't change, so it should be really easy to set them automatically.

@JuArceJuArce mentioned this pull request Nov 13, 2024
1 task
Base automatically changed from 1344-ci-improve-servers-setup to stagingNovember 13, 2024 17:40
# Conflicts:
#	Makefile
#	infra/ansible/README.md
#	infra/ansible/playbooks/batcher.yaml
#	infra/ansible/playbooks/docker.yaml
#	infra/ansible/playbooks/ini/config-batcher.ini.example
#	infra/ansible/playbooks/operator.yaml
#	infra/ansible/playbooks/templates/config-files/config-batcher.yaml.j2
#	infra/ansible/stage_inventory.yaml
#	infra/services/avs-sync.service
#	infra/services/explorer.service
#	infra/services/operator-1.service
#	infra/services/prometheus.service
#	infra/services/telemetry.service
#	infra/services/tracker.service
@JuArce
JuArce enabled auto-merge November 13, 2024 18:15
@JuArce
JuArce added this pull request to the merge queueNov 13, 2024
Merged via the queue into staging with commit c99d3a3Nov 13, 2024
@JuArce
JuArce deleted the operator_ansible branch November 13, 2024 19:14
PatStiles pushed a commit that referenced this pull request Nov 14, 2024
Co-authored-by: Klaus Lungwitz <klaus.lungwitz@lambdaclass.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@JuArce@klaus993@MarcosNicolau