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

Add note about cronjob #4657

Merged

Conversation

davidtrussler
Copy link
Contributor

This change adds a note to the "Signon API user token expires soon" section to the effect that any alerts relating to Staging (currently) and Integration (in the future) do not need any action. This is because the overnight cronjob updates the token in this environment to match any changes made in Production. The change is shown in the screenshot below.

Current Updated
Screenshot 2024-05-17 at 12 30 47 Screenshot 2024-05-17 at 12 30 18

Copy link
Contributor

@gclssvglx gclssvglx left a comment

Choose a reason for hiding this comment

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

Nice job @davidtrussler 🥇

@ChrisBAshton
Copy link
Contributor

ChrisBAshton commented May 17, 2024

Nice one! We can probably also stop these alerts from firing in Staging altogether 🤔

@davidtrussler davidtrussler merged commit 732a29b into main May 17, 2024
1 check passed
@davidtrussler davidtrussler deleted the Update-section_Signon-API-user-token-expires-soon branch May 17, 2024 15:01
@ChrisBAshton
Copy link
Contributor

Now dropped from Staging, so we should consider revising this @davidtrussler 🙏
alphagov/govuk-helm-charts#2059

@davidtrussler
Copy link
Contributor Author

Now dropped from Staging, so we should consider revising this @davidtrussler 🙏 alphagov/govuk-helm-charts#2059

Cheers Chris, yes, I've passed that onto the next shift with an explanation of events.

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

3 participants