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.

Sign up for GitHub

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

Jump to bottom

keystone deployment fixes #14928

Open
wants to merge 10 commits into
base: develop
Choose a base branch
from
Open

keystone deployment fixes #14928

wants to merge 10 commits into from
+153 −71

Conversation

krehermann
Copy link
Contributor

@krehermann krehermann commented Oct 23, 2024 •
edited by jira bot
Loading

KS-486: fix fowarder signer address
KS-487 add support for aptos keys in ocr3 contract config
KS-488: support ocr3 contract config updates
and add more logging

Requires

Resolves

@krehermann krehermann changed the title hacking keystone deployment fixes Oct 23, 2024
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Oct 23, 2024 •
edited
Loading

AER Report: CI Core ran successfully ✅

aer_workflow , commit

AER Report: Operator UI CI ran successfully ✅

aer_workflow , commit

@krehermann krehermann marked this pull request as ready for review October 23, 2024 21:09
@krehermann krehermann requested a review from a team as a code owner October 23, 2024 21:09
bolekk
bolekk previously approved these changes Oct 23, 2024
View reviewed changes
integration-tests/deployment/keystone/changeset/deploy_ocr3.go Outdated Show resolved Hide resolved
integration-tests/deployment/keystone/deploy.go Outdated Show resolved Hide resolved
integration-tests/deployment/keystone/types.go Outdated Show resolved Hide resolved
@cl-sonarqube-production
Copy link

Quality Gate passed Quality Gate passed

Issues
4 New issues
1 Fixed issue
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarQube

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bolekk bolekk bolekk left review comments

@vyzaldysanchez vyzaldysanchez Awaiting requested review from vyzaldysanchez

@patrickhuie19 patrickhuie19 Awaiting requested review from patrickhuie19

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@krehermann @bolekk