Skip to content

Create Smoke Test for DON Time #19031

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
Merged
DylanTinianov merged 118 commits into from
Sep 9, 2025
Merged

Create Smoke Test for DON Time #19031

DylanTinianov merged 118 commits into from
Sep 9, 2025

Conversation

@DylanTinianov
Copy link
Contributor

@DylanTinianov DylanTinianov commented Aug 20, 2025 •
edited
Loading

Created Smoke test on Local CRE Environment to verify identical consensus on DON Times in workflows. Also ensures don times are increasing.

@DylanTinianov
Hook donTimeStore
4d73ff7
@DylanTinianov
Update time_provider.go
60df926
@DylanTinianov
Merge branch 'develop' of https://github.com/smartcontractkit/chainlink…
b84f07a 
… into CAPPL-903-time-module-hook
@DylanTinianov
Bump chainlink-common
e2a125a
@DylanTinianov
Update time_provider.go
dd18ecc
@DylanTinianov
Update comments
f59a170
@DylanTinianov
Update time_provider.go
031fddb
@DylanTinianov
Add ctx to GetDONTime
16694b0
@DylanTinianov
Error on failed DON Time call
fa9f058
@DylanTinianov
Initialize dontimeStore
41ffc17
@DylanTinianov
Merge branch 'develop' of https://github.com/smartcontractkit/chainlink…
e575c3e 
… into CAPPL-903-time-module-hook
@DylanTinianov
Bump common
4af9ee6
@DylanTinianov
Bump common
827d57d
@DylanTinianov
Fix ambiguous selector
20a059f
@DylanTinianov
Add provider
c9b5e29
@DylanTinianov
Merge branch 'develop' of https://github.com/smartcontractkit/chainlink…
35ad4b6 
… into CAPPL-903-time-module-hook
@DylanTinianov
Bump
2bcea3f
@DylanTinianov
lint
34b3433
@DylanTinianov
Bump common
54981fa
@DylanTinianov
Merge branch 'develop' of https://github.com/smartcontractkit/chainlink…
2175eab 
… into CAPPL-903-time-module-hook
@DylanTinianov
Bump common
fd8a495
@DylanTinianov
Add dontime store
9673366
@DylanTinianov
Add dontime store
40d06ca
@DylanTinianov
Merge branch 'develop' of https://github.com/smartcontractkit/chainlink…
080659f 
… into CAPPL-903-time-module-hook
@DylanTinianov
Bump common
11f9bcc
@DylanTinianov
Bump common
caa6c9d
@DylanTinianov
Merge branch 'develop' of https://github.com/smartcontractkit/chainlink…
31d5c8f 
… into CAPPL-903-time-module-hook
@DylanTinianov
Bump common
3317ff5
@DylanTinianov
Initialize TimeProvider
5404f5a
@DylanTinianov
Bump common
d5e0ff2
@DylanTinianov DylanTinianov marked this pull request as ready for review September 9, 2025 18:12
@DylanTinianov DylanTinianov requested review from a team as code owners September 9, 2025 18:12
@DylanTinianov DylanTinianov requested a review from nolag September 9, 2025 18:12
@DylanTinianov DylanTinianov requested a review from nolag September 9, 2025 19:45
@cl-sonarqube-production
Copy link

cl-sonarqube-production bot commented Sep 9, 2025

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

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

See analysis details on SonarQube

@DylanTinianov DylanTinianov added this pull request to the merge queue Sep 9, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to a conflict with the base branch Sep 9, 2025
@DylanTinianov DylanTinianov added this pull request to the merge queue Sep 9, 2025
Merged via the queue into develop with commit 3b53839 Sep 9, 2025
178 of 179 checks passed
@DylanTinianov DylanTinianov deleted the CAPPL-1028-dontime-smoke-test branch September 9, 2025 23:03
This was referenced Sep 8, 2025
Draft
Draft
Draft
Draft
@github-actions github-actions bot mentioned this pull request Jul 25, 2025
Merged
@github-actions github-actions bot mentioned this pull request Oct 6, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vyzaldysanchez vyzaldysanchez vyzaldysanchez approved these changes

@krehermann krehermann krehermann approved these changes

@skudasov skudasov Awaiting requested review from skudasov

@Atrax1 Atrax1 Awaiting requested review from Atrax1

@nolag nolag Awaiting requested review from nolag

@MStreet3 MStreet3 Awaiting requested review from MStreet3

Assignees

@DylanTinianov DylanTinianov

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@DylanTinianov @nolag @vyzaldysanchez @krehermann