Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: use the cl as the default validator image if none are defined #503

Merged
merged 4 commits into from
Feb 29, 2024

Conversation

barnabasbusa
Copy link
Contributor

@barnabasbusa barnabasbusa commented Feb 29, 2024

This PR ensures that the validator image will be the same as CL image (if one is defined) and only fallback to the default validator image if there is no validator image nor beacon image is defined.

@barnabasbusa
Copy link
Contributor Author

barnabasbusa commented Feb 29, 2024

Gotta merge this: ethpandaops/eth-client-docker-image-builder#134 to pass all the CI jobs.

Have to rename all references from nimbus to nimbus-eth2 where the panda images are used.

Removed all panda references from CI jobs

@barnabasbusa barnabasbusa enabled auto-merge (squash) February 29, 2024 16:55
@barnabasbusa barnabasbusa merged commit 181dd04 into main Feb 29, 2024
16 of 17 checks passed
@barnabasbusa barnabasbusa deleted the bbusa/default-validator-image branch February 29, 2024 17:21
barnabasbusa pushed a commit that referenced this pull request Mar 8, 2024
🤖 I have created a release *beep* *boop*
---


##
[2.0.0](1.4.0...2.0.0)
(2024-03-08)


### ⚠ BREAKING CHANGES

* participant_network & rename participant fields.
([#508](#508))
* add node selectors features
([#491](#491))

### Features

* add keymanager to all validator processes
([#502](#502))
([836eda4](836eda4))
* add nimbus-eth1
([#496](#496))
([d599729](d599729))
* add node selectors features
([#491](#491))
([316d42f](316d42f))
* allow more detailed additional test configurations in assertoor_params
([#498](#498))
([fe2de7e](fe2de7e))
* enable api in assertoor config
([#495](#495))
([9ceae9c](9ceae9c))
* enable dencun-genesis
([#500](#500))
([beb764f](beb764f))
* make snapshot url configurable
([#507](#507))
([6fa0475](6fa0475))
* parameterize mev-boost args
([#400](#400))
([e48483a](e48483a))
* separate validator clients from CL clients
([#497](#497))
([90da2c3](90da2c3))


### Bug Fixes

* fix end index in validator ranges file
([#509](#509))
([da55be8](da55be8))
* lh vc flag logic
([#506](#506))
([bc5e725](bc5e725))
* nimbus-eth1 advertise proper extip
([#501](#501))
([1d5a779](1d5a779))
* README global node selector
([#504](#504))
([f9343a2](f9343a2))
* use the cl as the default validator image if none are defined
([#503](#503))
([181dd04](181dd04))


### Code Refactoring

* participant_network & rename participant fields.
([#508](#508))
([fab341b](fab341b))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to 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.

2 participants