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

Move deprecated evm versions to nightly CI workflow #14710

Merged
merged 1 commit into from
Mar 8, 2024

Conversation

r0qs
Copy link
Member

@r0qs r0qs commented Nov 29, 2023

As recommended in the discussion here: #14705 (comment), this PR moves the soltest to run exclusively on nightly jobs for Homestead and Byzantium EVM versions, which are now deprecated.

@r0qs r0qs self-assigned this Nov 29, 2023
matheusaaguiar
matheusaaguiar previously approved these changes Nov 29, 2023
Copy link
Collaborator

@matheusaaguiar matheusaaguiar left a comment

Choose a reason for hiding this comment

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

lgtm

@ekpyron
Copy link
Member

ekpyron commented Dec 4, 2023

@cameel is more of an expert for the CI intricacies here than I am, so maybe we let him give final approval here - I'd merge it myself :-).

@ekpyron ekpyron requested a review from cameel December 4, 2023 16:23
cameel
cameel previously approved these changes Dec 4, 2023
Copy link
Member

@cameel cameel left a comment

Choose a reason for hiding this comment

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

I have some suggestions but nothing critical. Looks fine overall.

.circleci/config.yml Outdated Show resolved Hide resolved
.circleci/config.yml Show resolved Hide resolved
.circleci/config.yml Outdated Show resolved Hide resolved
@cameel
Copy link
Member

cameel commented Dec 4, 2023

One more thing. Maybe you could also create a separate nightly-ossfuzz workflow and move t_ubu_ossfuzz to it? This way we could re-enable failure notifications for the other nightly jobs. Currently we're ignoring them all because ossfuzz always fails so won't really notice if they start failing too.

I'd do that in a separate PR though.

@r0qs r0qs dismissed stale reviews from cameel and matheusaaguiar via 1b42b1d December 4, 2023 20:34
@r0qs r0qs force-pushed the update-ci-for-evm-deprecations branch from a9df48b to a502d4d Compare December 6, 2023 15:05
@r0qs
Copy link
Member Author

r0qs commented Dec 6, 2023

One more thing. Maybe you could also create a separate nightly-ossfuzz workflow and move t_ubu_ossfuzz to it? This way we could re-enable failure notifications for the other nightly jobs. Currently we're ignoring them all because ossfuzz always fails so won't really notice if they start failing too.

Right. But isn't it a matter to just add the step matrix_notify_failure_unless_pr to all nightly jobs except t_ubu_ossfuzz? Why do we need a new workflow? Some of the nightly jobs even already have such step.

@cameel
Copy link
Member

cameel commented Dec 6, 2023

Oh, so we still have gitter notifications for some of this?

I was thinking more about the failure notification e-mail I get from nightly every day. Now I have to always check it to see if there were any other failing jobs, which is mildly annoying. If ossfuzz was in a separate workflow, I could just add a filter to ignore it and only care about notifications about the other workflow (which I would now only be receiving when it really fails).

I mean, if we had consistent gitter notifications for all nightly jobs maybe I could ignore these mails altogether. But my impression was that not all jobs have them.

@github-actions github-actions bot added the stale The issue/PR was marked as stale because it has been open for too long. label Dec 21, 2023
@r0qs r0qs removed the stale The issue/PR was marked as stale because it has been open for too long. label Dec 27, 2023
@github-actions github-actions bot added the stale The issue/PR was marked as stale because it has been open for too long. label Jan 13, 2024
@r0qs r0qs removed the stale The issue/PR was marked as stale because it has been open for too long. label Jan 16, 2024
@ethereum ethereum deleted a comment from github-actions bot Jan 16, 2024
@ethereum ethereum deleted a comment from github-actions bot Jan 16, 2024
@r0qs r0qs force-pushed the update-ci-for-evm-deprecations branch from a502d4d to c2615b2 Compare January 25, 2024 05:46
@r0qs r0qs force-pushed the update-ci-for-evm-deprecations branch from c2615b2 to a356764 Compare February 2, 2024 13:51
@r0qs
Copy link
Member Author

r0qs commented Feb 2, 2024

Oh, so we still have gitter notifications for some of this?

I was thinking more about the failure notification e-mail I get from nightly every day. Now I have to always check it to see if there were any other failing jobs, which is mildly annoying. If ossfuzz was in a separate workflow, I could just add a filter to ignore it and only care about notifications about the other workflow (which I would now only be receiving when it really fails).

I mean, if we had consistent gitter notifications for all nightly jobs maybe I could ignore these mails altogether. But my impression was that not all jobs have them.

I see, I don't know why I don't receive such notifications though, it might be some github config on my side, I will check it. Anyway, done here: #14831

@r0qs r0qs force-pushed the update-ci-for-evm-deprecations branch 2 times, most recently from 4417a67 to e3f46b1 Compare February 2, 2024 16:13
@github-actions github-actions bot added the stale The issue/PR was marked as stale because it has been open for too long. label Feb 17, 2024
@cameel cameel removed the stale The issue/PR was marked as stale because it has been open for too long. label Feb 18, 2024
@r0qs r0qs force-pushed the update-ci-for-evm-deprecations branch from e3f46b1 to b21f536 Compare February 18, 2024 21:31
@ethereum ethereum deleted a comment from github-actions bot Feb 18, 2024
@r0qs r0qs force-pushed the update-ci-for-evm-deprecations branch from b21f536 to 85a4406 Compare February 22, 2024 12:49
@r0qs r0qs requested a review from cameel February 22, 2024 13:00
@r0qs r0qs requested a review from matheusaaguiar February 22, 2024 13:00
matheusaaguiar
matheusaaguiar previously approved these changes Feb 22, 2024
@github-actions github-actions bot added the stale The issue/PR was marked as stale because it has been open for too long. label Mar 8, 2024
@cameel cameel force-pushed the update-ci-for-evm-deprecations branch from 85a4406 to 041b2c8 Compare March 8, 2024 16:08
@cameel cameel force-pushed the update-ci-for-evm-deprecations branch from 041b2c8 to 8644408 Compare March 8, 2024 16:15
@cameel cameel enabled auto-merge March 8, 2024 16:22
@cameel cameel removed the stale The issue/PR was marked as stale because it has been open for too long. label Mar 8, 2024
@ethereum ethereum deleted a comment from github-actions bot Mar 8, 2024
@cameel
Copy link
Member

cameel commented Mar 8, 2024

I see, I don't know why I don't receive such notifications though, it might be some github config on my side, I will check it.

I think it's in CircleCI settings. There's a page that lists the repositories you have access to and for each one you can select if you follow it. Then CircleCI sends you a notification about failed workflows. I get them for any PR where I pushed some commits and for the nightly runs.

@cameel cameel merged commit a2739f9 into develop Mar 8, 2024
74 of 75 checks passed
@cameel cameel deleted the update-ci-for-evm-deprecations branch March 8, 2024 16:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants