Skip to content
This repository has been archived by the owner on Jan 29, 2024. It is now read-only.

Added a quick blurb to let folks know where they can check pending maintenance updates #2115

Merged
merged 5 commits into from
Sep 5, 2023

Conversation

josh-martin-aiven
Copy link
Contributor

What changed, and why it matters

After a recent case with a customer, I noticed that it would be helpful to include some links in our documentation on maintenance updates. These links show customers where they can check their service for pending maintenance updates.

I think it's important to have something like this in the maintenance docs, as it is not readily apparent or searchable how to get this information from a customer perspective.

I tried to keep this update concise, and focused on directing customers to existing documentation on APIs as opposed to putting direct usage info on this doc.

Let me know your thoughts on this addition. I'm happy to discuss any and all feedback. :)

Reference links

Support task Jira:
https://aiven.atlassian.net/browse/SUP-717

Internal case study:
https://aiven.slab.com/posts/where-can-customers-check-for-pending-maintenance-updates-00047810-j9zh17em

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Aug 28, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: bf359bf
Status: ✅  Deploy successful!
Preview URL: https://f369e441.devportal.pages.dev
Branch Preview URL: https://check-maintenance-updates-ti.devportal.pages.dev

View logs

@wojcik-dorota wojcik-dorota self-requested a review August 30, 2023 19:02
@wojcik-dorota wojcik-dorota self-assigned this Aug 30, 2023
Copy link
Contributor

@wojcik-dorota wojcik-dorota left a comment

Choose a reason for hiding this comment

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

This is very helpful content, thanks for adding this! A few editorial remarks.

Security updates, platform updates that affect reliability or stability of the service nodes, and quarterly patch updates are always mandatory. Other updates are initially optional. Advance notice is given for all updates. After optional updates have been available for six months, they become mandatory and are applied on the next week's maintenance window at the earliest. This means you have at least 7 days advance notice with exception of critical security updates. These critical updates are applied in the maintenance window of the current week. During service upgrades, maintenance updates are automatically applied and do not require any action from you.

Pending maintenance updates can be viewed in three places.
Copy link
Contributor

Choose a reason for hiding this comment

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

Please use the active voice (vs the passive voice) whenever possible:
You can view maintenance updates pending for your service using the following:

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Doneso. :)

Pending maintenance updates can be viewed in three places.

1. Service overview page in `the Aiven Console <https://console.aiven.io/>`_.
Copy link
Contributor

Choose a reason for hiding this comment

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

Please

  • Rephrase to Service's **Overview** page in `Aiven Console <https://console.aiven.io/>`_ (GUI items always within **)
  • Use the unordered list (bullets) (the order of the items in the list is not significant).
  • Remove the full stops at the ends of all three list items (the list items are not full sentences).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Also done.

Also, I finally found the style guide. I'll make sure to review this for future docs updates. If this isn't the right style guide, lemme know. :)
https://aiven.slab.com/posts/style-guide-resources-for-editors-dag2nrwl

Copy link
Contributor

Choose a reason for hiding this comment

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

Reworded addition to use active voice. 
put emphasis on overview. 
changed numbered list to bullet list.
Security updates, platform updates that affect reliability or stability of the service nodes, and quarterly patch updates are always mandatory. Other updates are initially optional. Advance notice is given for all updates. After optional updates have been available for six months, they become mandatory and are applied on the next week's maintenance window at the earliest. This means you have at least 7 days advance notice with exception of critical security updates. These critical updates are applied in the maintenance window of the current week. During service upgrades, maintenance updates are automatically applied and do not require any action from you.

You can view pending maintenance updates in three places.
Copy link
Contributor

Choose a reason for hiding this comment

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

Please rephrase to "You can view maintenance updates pending for your service using the following:".
A command or an API endpoint is not a place :-)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Got it. Just updated the wording to match latest suggestions. Let me know if there's anything else you'd like to adjust, discuss. :)

Copy link
Contributor

@wojcik-dorota wojcik-dorota left a comment

Choose a reason for hiding this comment

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

Thanks! a few more details :-)

You can view pending maintenance updates in three places.

- Service **overview** page in `the Aiven Console <https://console.aiven.io/>`_
Copy link
Contributor

Choose a reason for hiding this comment

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

Please use Service's **Overview** page in `Aiven Console <https://console.aiven.io/>`_

Updated wording of initial sentence on bottom section. 
Used possessive form for for service's overview page. 
Removed the word "the" in hyperlinks.
Copy link
Contributor

@wojcik-dorota wojcik-dorota left a comment

Choose a reason for hiding this comment

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

Thanks for all the updates! LGTM now :-)

@wojcik-dorota wojcik-dorota merged commit 8a6da60 into main Sep 5, 2023
@wojcik-dorota wojcik-dorota deleted the check-maintenance-updates-tip branch September 5, 2023 14:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants