Skip to content

ci: add yaml-lint GitHub Workflow#368

Merged
mergify[bot] merged 4 commits into
csi-addons:mainfrom
nixpanic:workflow/yamllint
Jun 16, 2023
Merged

ci: add yaml-lint GitHub Workflow#368
mergify[bot] merged 4 commits into
csi-addons:mainfrom
nixpanic:workflow/yamllint

Conversation

@nixpanic

Copy link
Copy Markdown
Member

Initial yamllint validation for all GitHub Workflows.

Updates: #12

nixpanic added 2 commits June 14, 2023 13:13
Initial `yamllint` validation for all GitHub Workflows.

Updates: #12
Signed-off-by: Niels de Vos <ndevos@ibm.com>
Many yaml files have this warning in `yamllint`:

    [warning] missing document start "---" (document-start)

Added the `---` document start to all files that did not have it yet.

Signed-off-by: Niels de Vos <ndevos@ibm.com>
@nixpanic nixpanic marked this pull request as ready for review June 15, 2023 12:46
@mergify mergify Bot requested review from Rakshith-R and yati1998 June 15, 2023 12:46
@nixpanic nixpanic force-pushed the workflow/yamllint branch from dd1f9f4 to 50a14d4 Compare June 15, 2023 13:02
nixpanic added 2 commits June 15, 2023 15:11
Signed-off-by: Niels de Vos <ndevos@ibm.com>
Signed-off-by: Niels de Vos <ndevos@ibm.com>
@nixpanic nixpanic force-pushed the workflow/yamllint branch from 50a14d4 to d817579 Compare June 15, 2023 13:11
@nixpanic

Copy link
Copy Markdown
Member Author

@Rakshith-R @yati1998 this is really ready for review now.

@Rakshith-R Rakshith-R left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM thanks !

@mergify mergify Bot merged commit ef6efbd into csi-addons:main Jun 16, 2023
black-dragon74 pushed a commit to black-dragon74/kubernetes-csi-addons that referenced this pull request Jan 13, 2026
DFBUGS-5055: [release-4.19] csiaddonsnode: Add retry with exponential backoff for connections
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