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

Compute budget reimbursement + configuration #430

Merged
merged 14 commits into from
Feb 15, 2023

Conversation

aalu1418
Copy link
Collaborator

@aalu1418 aalu1418 commented Nov 22, 2022

Combining PRs

Note: CL nodes shouldn't upgrade until the program is first upgraded.
#405 (comment)

E2E CI won't pass because of required changes on the core side - corresponding changes: smartcontractkit/chainlink#7943

@aalu1418 aalu1418 temporarily deployed to integration November 22, 2022 16:25 Inactive
@aalu1418 aalu1418 temporarily deployed to integration November 22, 2022 16:25 Inactive
@archseer archseer temporarily deployed to integration February 9, 2023 06:51 — with GitHub Actions Inactive
@archseer archseer temporarily deployed to integration February 9, 2023 06:52 — with GitHub Actions Inactive
@aalu1418 aalu1418 temporarily deployed to integration February 14, 2023 14:20 — with GitHub Actions Inactive
@aalu1418 aalu1418 temporarily deployed to integration February 14, 2023 14:20 — with GitHub Actions Inactive
@aalu1418 aalu1418 temporarily deployed to integration February 14, 2023 14:20 — with GitHub Actions Inactive
@github-actions
Copy link
Contributor

github-actions bot commented Feb 14, 2023

Smoke Test Results

0 tests   - 1   0 ✔️ ±0   0s ⏱️ - 6m 44s
0 suites  - 1   0 💤 ±0 
0 files    - 1   0  - 1 

Results for commit 7d48285. ± Comparison against base commit 7c09106.

♻️ This comment has been updated with latest results.

@aalu1418 aalu1418 temporarily deployed to integration February 14, 2023 14:37 — with GitHub Actions Inactive
@aalu1418 aalu1418 temporarily deployed to integration February 14, 2023 14:37 — with GitHub Actions Inactive
@aalu1418 aalu1418 temporarily deployed to integration February 14, 2023 14:37 — with GitHub Actions Inactive
@aalu1418 aalu1418 temporarily deployed to integration February 14, 2023 14:53 — with GitHub Actions Inactive
@aalu1418 aalu1418 temporarily deployed to integration February 14, 2023 14:53 — with GitHub Actions Inactive
@aalu1418 aalu1418 temporarily deployed to integration February 14, 2023 14:53 — with GitHub Actions Inactive
@aalu1418 aalu1418 temporarily deployed to integration February 14, 2023 15:09 — with GitHub Actions Inactive
@aalu1418 aalu1418 temporarily deployed to integration February 14, 2023 15:09 — with GitHub Actions Inactive
@aalu1418 aalu1418 temporarily deployed to integration February 14, 2023 15:09 — with GitHub Actions Inactive
@aalu1418 aalu1418 temporarily deployed to integration February 14, 2023 15:49 — with GitHub Actions Inactive
@aalu1418 aalu1418 temporarily deployed to integration February 14, 2023 15:50 — with GitHub Actions Inactive
@aalu1418 aalu1418 temporarily deployed to integration February 14, 2023 15:55 — with GitHub Actions Inactive
@aalu1418 aalu1418 temporarily deployed to integration February 14, 2023 15:59 — with GitHub Actions Inactive
@aalu1418 aalu1418 temporarily deployed to integration February 14, 2023 15:59 — with GitHub Actions Inactive
@aalu1418 aalu1418 temporarily deployed to integration February 14, 2023 15:59 — with GitHub Actions Inactive
@aalu1418 aalu1418 temporarily deployed to integration February 14, 2023 16:05 — with GitHub Actions Inactive
@aalu1418 aalu1418 temporarily deployed to integration February 14, 2023 16:12 — with GitHub Actions Inactive
@aalu1418
Copy link
Collaborator Author

Had to manually run the smoke test CI because of the breaking config interface changes that are not in core develop.

Here's the custom run: https://github.com/smartcontractkit/chainlink-solana/actions/runs/4175643462/jobs/7231139217
It uses this branch + this commit (4c8245ae57e1f570328525c92120244faec1c090) from smartcontractkit/chainlink#7943

image

@aalu1418
Copy link
Collaborator Author

note: merging without e2e test passing because of corresponding changes needed in core. however the test do pass when run against the correct branch (as shown above)

@aalu1418 aalu1418 merged commit c20b570 into develop Feb 15, 2023
@aalu1418 aalu1418 deleted the compute-budget-and-config branch February 15, 2023 16:41
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