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

CLOUDP-278929: Send a remind if an API version is approching the release date #260

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

andreaangiolillo
Copy link
Collaborator

Proposed changes

This PR introduces a new GitHub Action that runs daily to verify that the API versions listed in the versions.json file in the development environment are not nearing their release dates (less than three weeks away). If there are any upcoming API versions scheduled for release, the action sends a notification to the team’s Slack channel.

Jira ticket: CLOUDP-278929

@andreaangiolillo andreaangiolillo marked this pull request as ready for review October 17, 2024 17:30
@andreaangiolillo andreaangiolillo requested a review from a team as a code owner October 17, 2024 17:30
done

if [[ ${#dates_within_3_weeks[@]} -gt 0 ]]; then
echo "API Versions that will be release in the next 3 weeks: ${dates_within_3_weeks[*]}"
Copy link
Collaborator

Choose a reason for hiding this comment

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

this will get quite noisy (every day, for 3 weeks in a row, we'll get this notification and tag @apix-2-on-call).
Is it too complicated to have this trigger conditions?

  • isMonday and an API version is going to be released withing the next 3 weeks
  • isWeekday and an API version is going to be released withing the 6 days (this is the release week)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

isWeekday and an API version is going to be released withing the 6 days (this is the release week)

Isn't this still too noisy during the release week? I think that having the message only on Monday should be enough. WDYT?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

updated

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