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

ci: Disable daily checking for version but not security updates #500

Merged
merged 2 commits into from
Jun 7, 2023

Conversation

trishankatdatadog
Copy link
Member

Disable daily checking for version but not security updates.

Maybe we should also document a process for version updates when cutting a new release?

@trishankatdatadog trishankatdatadog changed the title Disable daily checking for version but not security updates ci: Disable daily checking for version but not security updates Jun 2, 2023
Copy link
Contributor

@znewman01 znewman01 left a comment

Choose a reason for hiding this comment

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

Hm...I'm okay with this but I'm not 100% sure it actually saves time in the long run. I worry that when there's a security review, we'll have to deal with a bunch of accumulated changes.

@rdimitrov
Copy link
Contributor

I think that if we disable version checks we may have API changes that can make it harder to effectively address the security updates. Perhaps we can address this by changing the version checks schedule to weekly or bi-weekly, WDYT? Personally, I've configured this to be a weekly job for go-tuf-metadata, at least for the time being.

@trishankatdatadog
Copy link
Member Author

I'm good with weekly. If you both agree, I can make that happen here

Disable daily checking for version but not security updates.

Maybe we should also document a process for version updates when cutting a new release?

Signed-off-by: Trishank Karthik Kuppusamy <trishank.kuppusamy@datadoghq.com>
Copy link
Contributor

@rdimitrov rdimitrov left a comment

Choose a reason for hiding this comment

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

Having it on a weekly basis sounds great for me! 👍

@znewman01
Copy link
Contributor

Weekly works for me too!

Signed-off-by: Trishank Karthik Kuppusamy <trishank.kuppusamy@datadoghq.com>
@znewman01 znewman01 merged commit 0bf668e into master Jun 7, 2023
@znewman01 znewman01 deleted the trishankatdatadog-patch-1 branch June 7, 2023 15:46
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.

3 participants