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

chore: bump kurtosis-cdk version to support pre-deployed gas token #202

Merged

Conversation

leovct
Copy link
Member

@leovct leovct commented Nov 27, 2024

Description

Related to 0xPolygon/kurtosis-cdk#386

It looks like the CI doesn't run the e2e tests on my PR, maybe because it comes from an external fork? I've created a local PR on my fork to make sure they all pass. Note that the fork12-pp test doesn't pass on the develop branch either.

Screenshot 2024-11-27 at 10 48 44

@leovct leovct marked this pull request as ready for review November 27, 2024 09:50
@Stefan-Ethernal
Copy link
Collaborator

@leovct the PP E2E tests were fixed through #198, although it has not landed to the develop branch yet.

Copy link
Contributor

@vcastellm vcastellm left a comment

Choose a reason for hiding this comment

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

Nice!

@leovct
Copy link
Member Author

leovct commented Nov 27, 2024

I'm afraid the SonarCloud job won't pass because I'm opening a PR from an external fork. Is this a blocker for the PR to be merged?

@Stefan-Ethernal
Copy link
Collaborator

I'm afraid the SonarCloud job won't pass because I'm opening a PR from an external fork. Is this a blocker for the PR to be merged?

Nope, it is a known issue and you have not affected the source code, therefore it could not have negatively impacted the SonarCloud.

@Stefan-Ethernal Stefan-Ethernal merged commit 1fe7a4a into 0xPolygon:develop Nov 27, 2024
4 of 5 checks passed
@leovct leovct deleted the chore/kurtosis-cdk-bump-gas-token branch November 28, 2024 11:05
joanestebanr pushed a commit that referenced this pull request Nov 28, 2024
)

* chore: kurtosis-cdk bump (gas token update)

* test

* chore: use new kurtosis-cdk tag

* chore: nit

* chore: clean up
joanestebanr added a commit that referenced this pull request Dec 2, 2024
… certificates in a history table (#208)

- Add new field `retries` to database, that keep the count of times of regenerated the certificate
- The discarded certificates are move (if configuration allow that) to a new table `certificate_info_history`
- Cherry-picked #202 to fix e2e-test

## Configuration
```
[AggSender]
KeepCertificatesHistory = true
```
---------

Co-authored-by: Léo Vincent <[email protected]>
Co-authored-by: Toni Ramírez <[email protected]>
Co-authored-by: Stefan Negovanović <[email protected]>
ToniRamirezM pushed a commit that referenced this pull request Dec 3, 2024
)

* chore: kurtosis-cdk bump (gas token update)

* test

* chore: use new kurtosis-cdk tag

* chore: nit

* chore: clean up
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.

3 participants