Browse Source

Clean up YAML files in the repository. (#11570)

* Assorted yamllint config updates.

* Extended line length limit to 150. This is still a reasonable limit
  but cuts down on warnings a bit.
* Added explicit ignore directives for external repo paths. This makes
  using yamllint locally nicer.
* Enabled the rule to require null values to be explicit. This rule
  helps prevent accidental bogus mapping problems resulting from
  implicit null values combined with incorrect indentation.

This only results in new errors in a couple of GHA workflows, which will
be fixed in the next commit.

* Fix all yamllint warnings in yaml file sin the repo.

Most are indentation or spurious empty lines.

* Fix invalid null value.
Austin S. Hemmelgarn 3 years ago
parent
commit
f183dc8aea

+ 5 - 6
.codeclimate.yml

@@ -43,11 +43,11 @@ plugins:
     enabled: false
     config:
       languages:
-      - javascript:
-          mass_threshold: 100
-      - python:
-          python_version: 3
-          mass_threshold: 100
+        - javascript:
+            mass_threshold: 100
+        - python:
+            python_version: 3
+            mass_threshold: 100
     checks:
       Similar code:
         enabled: false
@@ -96,4 +96,3 @@ exclude_patterns:
   - "collectors/node.d.plugin/node_modules/extend.js"
   - "collectors/node.d.plugin/node_modules/pixl-xml.js"
   - "collectors/node.d.plugin/node_modules/net-snmp.js"
-

+ 1 - 1
.github/workflows/build-and-install.yml

@@ -4,7 +4,7 @@ on:
   push:
     branches:
       - master
-  pull_request:
+  pull_request: null
 env:
   DO_NOT_TRACK: 1
 jobs:

+ 1 - 1
.github/workflows/checks.yml

@@ -4,7 +4,7 @@ on:
   push:
     branches:
       - master
-  pull_request:
+  pull_request: null
 env:
   DO_NOT_TRACK: 1
 jobs:

+ 1 - 1
.github/workflows/coverity.yml

@@ -40,7 +40,7 @@ jobs:
         uses: rtCamp/action-slack-notify@v2
         env:
           SLACK_COLOR: 'danger'
-          SLACK_FOOTER:
+          SLACK_FOOTER: ''
           SLACK_ICON_EMOJI: ':github-actions:'
           SLACK_TITLE: 'Coverity run failed:'
           SLACK_USERNAME: 'GitHub Actions'

+ 2 - 2
.github/workflows/docker.yml

@@ -4,7 +4,7 @@ on:
   push:
     branches:
       - master
-  pull_request:
+  pull_request: null
   workflow_dispatch:
     inputs:
       version:
@@ -99,7 +99,7 @@ jobs:
         uses: rtCamp/action-slack-notify@v2
         env:
           SLACK_COLOR: 'danger'
-          SLACK_FOOTER:
+          SLACK_FOOTER: ''
           SLACK_ICON_EMOJI: ':github-actions:'
           SLACK_TITLE: 'Docker Build failed:'
           SLACK_USERNAME: 'GitHub Actions'

+ 0 - 1
.github/workflows/packaging.yml

@@ -133,7 +133,6 @@ jobs:
         uses: rtCamp/action-slack-notify@v2
         env:
           SLACK_COLOR: 'danger'
-          SLACK_FOOTER:
           SLACK_ICON_EMOJI: ':github-actions:'
           SLACK_TITLE: 'Package Build failed:'
           SLACK_USERNAME: 'GitHub Actions'

+ 2 - 2
.github/workflows/repoconfig-packages.yml

@@ -2,7 +2,7 @@
 # Handles building of binary packages for the agent.
 name: Repository Packages
 on:
-  workflow_dispatch:
+  workflow_dispatch: null
 env:
   DO_NOT_TRACK: 1
 jobs:
@@ -60,7 +60,7 @@ jobs:
         uses: rtCamp/action-slack-notify@v2
         env:
           SLACK_COLOR: 'danger'
-          SLACK_FOOTER:
+          SLACK_FOOTER: ''
           SLACK_ICON_EMOJI: ':github-actions:'
           SLACK_TITLE: 'Repository Package Build failed:'
           SLACK_USERNAME: 'GitHub Actions'

+ 1 - 1
.github/workflows/review.yml

@@ -2,7 +2,7 @@
 # Runs various ReviewDog based checks against PR with suggested changes to improve quality
 name: Review
 on:
-  pull_request:
+  pull_request: null
 env:
   run_eslint: 0
   run_hadolint: 0

+ 10 - 4
.yamllint.yml

@@ -6,10 +6,11 @@ yaml-files:
   - 'collectors/python.d.plugin/*.conf'
   - 'collectors/python.d.plugin/*/*.conf'
 
+ignore: |
+  mqtt_websockets/
+  packaging/makeself/tmp/
+
 rules:
-  line-length:
-    max: 120
-    level: warning
   braces: enable
   brackets: enable
   colons: enable
@@ -19,9 +20,14 @@ rules:
   document-end: disable
   document-start: disable
   empty-lines: enable
-  empty-values: disable
+  empty-values: enable
   hyphens: enable
   indentation: enable
+  line-length:
+    max: 150
+    level: warning
+    allow-non-breakable-words: true
+    allow-non-breakable-inline-mappings: true
   key-duplicates: enable
   key-ordering: disable
   new-line-at-end-of-file: enable

+ 3 - 4
build_external/scenarios/aclk-testing/agent-compose.yml

@@ -5,8 +5,8 @@ services:
       context: ../../..
       dockerfile: build_external/make-install.Dockerfile
       args:
-      - DISTRO=arch
-      - VERSION=current
+        - DISTRO=arch
+        - VERSION=current
     image: arch_current_dev:latest
     command: >
       sh -c "echo -n 00000000-0000-0000-0000-000000000000 >/var/lib/netdata/cloud.d/claimed_id &&
@@ -15,5 +15,4 @@ services:
              echo '  agent cloud link port = 9002' >>/etc/netdata/netdata.conf &&
              /usr/sbin/netdata -D"
     ports:
-    - 20000:19999
-
+      - 20000:19999

Some files were not shown because too many files changed in this diff