Skip to content

Add notes on non-retriable procedures#765

Merged
DmitriiRabenok merged 4 commits intomainfrom
doc/non-retriable-procedures
Aug 14, 2025
Merged

Add notes on non-retriable procedures#765
DmitriiRabenok merged 4 commits intomainfrom
doc/non-retriable-procedures

Conversation

@theboringstuff
Copy link
Collaborator

@theboringstuff theboringstuff commented Aug 14, 2025

Description

Currently, some procedures are in general non-retriable with the same "procedure.yaml", which may be not obvious for users. Particularly:

  • Upgrade is not retriable with the same "procedure.yaml" once it updates k8s version in "cluster.yaml".
  • Add node is not retriable with the same "procedure.yaml" once it completes with any task
  • Remove node is not retriable with the same "procedure.yaml" once it completes with any task

To retry such procedures, it is required to change "cluster.yaml" or "procedure.yaml" accordingly

Solution

  • Added notes on non-retriable procedures

Checklist

  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • There is no breaking changes, or migration patch is provided
  • Integration CI passed
  • Unit tests. If Yes list of new/changed tests with brief description
  • There is no merge conflicts

@theboringstuff theboringstuff changed the title added notes Add notes on non-retriable procedures Aug 14, 2025
@theboringstuff theboringstuff marked this pull request as ready for review August 14, 2025 07:15
@theboringstuff
Copy link
Collaborator Author

@disa1217 please review doc changes

Copy link
Collaborator

@disa1217 disa1217 left a comment

Choose a reason for hiding this comment

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

Reviewed.

@DmitriiRabenok DmitriiRabenok merged commit c8573bf into main Aug 14, 2025
29 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Aug 14, 2025
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.

6 participants