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

kernel-arch-complete.sh: Update email address for CIP notifications #138

Merged
merged 1 commit into from
Feb 22, 2024

Conversation

patersonc
Copy link
Contributor

CIP would like to move the KernelCI email notifications over to the cip-testing-results mailing list.

Signed-off-by: Chris Paterson <chris.paterson2@renesas.com>
@patersonc
Copy link
Contributor Author

It would be good to be able to test this change before it is merged, as I need to make sure that the emails from the KernelCI bot can get through without manual moderation. If they cannot, I may need to get a special target email address configured as we had before for the cip-dev mailing list.

@nuclearcat
Copy link
Collaborator

I think it is a bit difficult to get staging send email to regular targets.
I suggest we can try following: we can do usual monday production update, i will notify in Slack on completion, theb if everything will work as should - you confirm, and we keep it, if not, i can quickly reverse or update address - and do partial production update again.
Is it ok like this?

@patersonc
Copy link
Contributor Author

I think it is a bit difficult to get staging send email to regular targets. I suggest we can try following: we can do usual monday production update, i will notify in Slack on completion, theb if everything will work as should - you confirm, and we keep it, if not, i can quickly reverse or update address - and do partial production update again. Is it ok like this?

That's fine with me.

If it's easier, we can just merge the change. If it needs changing again then I'll create a new MR. CIP can live without the emails for a little while if needed.

@nuclearcat
Copy link
Collaborator

Ok merging and i will try to check after production update at monday if there is any errors or issues.

@nuclearcat nuclearcat merged commit 1f3fe8c into kernelci:main Feb 22, 2024
1 check passed
@patersonc patersonc deleted the patersonc/update-cip-email branch February 22, 2024 12:09
@patersonc
Copy link
Contributor Author

Hello

I'm still seeing emails sent to the old mailing list. Has this MR been pushed to production?

Thanks

@nuclearcat
Copy link
Collaborator

I will update it manually today, i forgot that legacy system doesn't update jenkins properly.

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.

None yet

2 participants