Browse Source

ci: fix super-linter configuration settings (#7708)

- Revert the version pin from #7706.
- Override the YAML linter config to be more forgiving.
- Update YAML lint warnings in a number of files.

The choice of which lints to fix and which to override was ad hoc: I fixed the ones that were mainly whitespace oriented, and suppressed the ones that were document-structure related.

Fixes #7707.
pull/7712/head
M. J. Fromberger 3 years ago
committed by GitHub
parent
commit
5eb50a43b5
No known key found for this signature in database GPG Key ID: 4AEE18F83AFDEB23
32 changed files with 53 additions and 81 deletions
  1. +10
    -10
      .github/auto-comment.yml
  2. +0
    -8
      .github/linter/markdownlint.yml
  3. +8
    -0
      .github/linters/markdownlint.yml
  4. +9
    -0
      .github/linters/yaml-lint.yml
  5. +3
    -3
      .github/workflows/docker.yml
  6. +2
    -2
      .github/workflows/e2e-nightly-34x.yml
  7. +2
    -2
      .github/workflows/e2e-nightly-35x.yml
  8. +2
    -2
      .github/workflows/e2e-nightly-master.yml
  9. +1
    -2
      .github/workflows/e2e.yml
  10. +1
    -1
      .github/workflows/fuzz-nightly.yml
  11. +1
    -1
      .github/workflows/linkchecker.yml
  12. +2
    -2
      .github/workflows/linter.yml
  13. +1
    -1
      .github/workflows/release.yml
  14. +5
    -5
      .goreleaser.yml
  15. +2
    -2
      docker-compose.yml
  16. +2
    -5
      networks/remote/ansible/config.yml
  17. +0
    -2
      networks/remote/ansible/install.yml
  18. +1
    -4
      networks/remote/ansible/logzio.yml
  19. +0
    -3
      networks/remote/ansible/reset.yml
  20. +0
    -2
      networks/remote/ansible/restart.yml
  21. +1
    -3
      networks/remote/ansible/roles/config/tasks/main.yml
  22. +0
    -2
      networks/remote/ansible/roles/install/handlers/main.yml
  23. +0
    -2
      networks/remote/ansible/roles/install/tasks/main.yml
  24. +0
    -2
      networks/remote/ansible/roles/logzio/handlers/main.yml
  25. +0
    -2
      networks/remote/ansible/roles/logzio/tasks/main.yml
  26. +0
    -2
      networks/remote/ansible/roles/start/tasks/main.yml
  27. +0
    -2
      networks/remote/ansible/roles/status/tasks/main.yml
  28. +0
    -2
      networks/remote/ansible/roles/stop/tasks/main.yml
  29. +0
    -1
      networks/remote/ansible/roles/unsafe_reset/tasks/main.yml
  30. +0
    -2
      networks/remote/ansible/start.yml
  31. +0
    -2
      networks/remote/ansible/status.yml
  32. +0
    -2
      networks/remote/ansible/stop.yml

+ 10
- 10
.github/auto-comment.yml View File

@ -1,16 +1,16 @@
pullRequestOpened: |
:wave: Thanks for creating a PR!
:wave: Thanks for creating a PR!
Before we can merge this PR, please make sure that all the following items have been
Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.
write a little note why.
- [ ] Wrote tests
- [ ] Updated CHANGELOG_PENDING.md
- [ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
- [ ] Updated relevant documentation (`docs/`) and code comments
- [ ] Re-reviewed `Files changed` in the Github PR explorer
- [ ] Applied Appropriate Labels
- [ ] Wrote tests
- [ ] Updated CHANGELOG_PENDING.md
- [ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
- [ ] Updated relevant documentation (`docs/`) and code comments
- [ ] Re-reviewed `Files changed` in the Github PR explorer
- [ ] Applied Appropriate Labels
Thank you for your contribution to Tendermint! :rocket:
Thank you for your contribution to Tendermint! :rocket:

+ 0
- 8
.github/linter/markdownlint.yml View File

@ -1,8 +0,0 @@
default: true,
MD007: { "indent": 4 }
MD013: false
MD024: { siblings_only: true }
MD025: false
MD033: { no-inline-html: false }
no-hard-tabs: false
whitespace: false

+ 8
- 0
.github/linters/markdownlint.yml View File

@ -0,0 +1,8 @@
default: true,
MD007: {"indent": 4}
MD013: false
MD024: {siblings_only: true}
MD025: false
MD033: {no-inline-html: false}
no-hard-tabs: false
whitespace: false

+ 9
- 0
.github/linters/yaml-lint.yml View File

@ -0,0 +1,9 @@
---
# Default rules for YAML linting from super-linter.
# See: See https://yamllint.readthedocs.io/en/stable/rules.html
extends: default
rules:
document-end: disable
document-start: disable
line-length: disable
truthy: disable

+ 3
- 3
.github/workflows/docker.yml View File

@ -1,13 +1,13 @@
name: Docker
# Build & Push rebuilds the tendermint docker image on every push to master and creation of tags
# Build & Push rebuilds the tendermint docker image on every push to master and creation of tags
# and pushes the image to https://hub.docker.com/r/interchainio/simapp/tags
on:
push:
branches:
- master
tags:
- "v[0-9]+.[0-9]+.[0-9]+" # Push events to matching v*, i.e. v1.0, v20.15.10
- "v[0-9]+.[0-9]+.[0-9]+-rc*" # Push events to matching v*, i.e. v1.0-rc1, v20.15.10-rc5
- "v[0-9]+.[0-9]+.[0-9]+" # Push events to matching v*, i.e. v1.0, v20.15.10
- "v[0-9]+.[0-9]+.[0-9]+-rc*" # Push events to matching v*, i.e. v1.0-rc1, v20.15.10-rc5
jobs:
build:


+ 2
- 2
.github/workflows/e2e-nightly-34x.yml View File

@ -6,7 +6,7 @@
name: e2e-nightly-34x
on:
workflow_dispatch: # allow running workflow manually, in theory
workflow_dispatch: # allow running workflow manually, in theory
schedule:
- cron: '0 2 * * *'
@ -59,7 +59,7 @@ jobs:
SLACK_MESSAGE: Nightly E2E tests failed on v0.34.x
SLACK_FOOTER: ''
e2e-nightly-success: # may turn this off once they seem to pass consistently
e2e-nightly-success: # may turn this off once they seem to pass consistently
needs: e2e-nightly-test
if: ${{ success() }}
runs-on: ubuntu-latest


+ 2
- 2
.github/workflows/e2e-nightly-35x.yml View File

@ -5,7 +5,7 @@
name: e2e-nightly-35x
on:
workflow_dispatch: # allow running workflow manually
workflow_dispatch: # allow running workflow manually
schedule:
- cron: '0 2 * * *'
@ -59,7 +59,7 @@ jobs:
SLACK_MESSAGE: Nightly E2E tests failed on v0.35.x
SLACK_FOOTER: ''
e2e-nightly-success: # may turn this off once they seem to pass consistently
e2e-nightly-success: # may turn this off once they seem to pass consistently
needs: e2e-nightly-test
if: ${{ success() }}
runs-on: ubuntu-latest


+ 2
- 2
.github/workflows/e2e-nightly-master.yml View File

@ -5,7 +5,7 @@
name: e2e-nightly-master
on:
workflow_dispatch: # allow running workflow manually
workflow_dispatch: # allow running workflow manually
schedule:
- cron: '0 2 * * *'
@ -56,7 +56,7 @@ jobs:
SLACK_MESSAGE: Nightly E2E tests failed on master
SLACK_FOOTER: ''
e2e-nightly-success: # may turn this off once they seem to pass consistently
e2e-nightly-success: # may turn this off once they seem to pass consistently
needs: e2e-nightly-test
if: ${{ success() }}
runs-on: ubuntu-latest


+ 1
- 2
.github/workflows/e2e.yml View File

@ -2,7 +2,7 @@ name: e2e
# Runs the CI end-to-end test network on all pushes to master or release branches
# and every pull request, but only if any Go files have been changed.
on:
workflow_dispatch: # allow running workflow manually
workflow_dispatch: # allow running workflow manually
pull_request:
push:
branches:
@ -35,4 +35,3 @@ jobs:
working-directory: test/e2e
run: ./run-multiple.sh networks/ci.toml
if: "env.GIT_DIFF != ''"

+ 1
- 1
.github/workflows/fuzz-nightly.yml View File

@ -1,7 +1,7 @@
# Runs fuzzing nightly.
name: Fuzz Tests
on:
workflow_dispatch: # allow running workflow manually
workflow_dispatch: # allow running workflow manually
schedule:
- cron: '0 3 * * *'
pull_request:


+ 1
- 1
.github/workflows/linkchecker.yml View File

@ -1,5 +1,5 @@
name: Check Markdown links
on:
on:
schedule:
- cron: '* */24 * * *'
jobs:


+ 2
- 2
.github/workflows/linter.yml View File

@ -21,12 +21,12 @@ jobs:
- name: Checkout Code
uses: actions/checkout@v2.4.0
- name: Lint Code Base
uses: docker://github/super-linter:v4.8.5
uses: docker://github/super-linter:v4
env:
LINTER_RULES_PATH: .
VALIDATE_ALL_CODEBASE: true
DEFAULT_BRANCH: master
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
VALIDATE_MD: true
VALIDATE_OPENAPI: true
VALIDATE_YAML: true
YAML_CONFIG_FILE: yaml-lint.yml

+ 1
- 1
.github/workflows/release.yml View File

@ -5,7 +5,7 @@ on:
branches:
- "RC[0-9]/**"
tags:
- "v[0-9]+.[0-9]+.[0-9]+" # Push events to matching v*, i.e. v1.0, v20.15.10
- "v[0-9]+.[0-9]+.[0-9]+" # Push events to matching v*, i.e. v1.0, v20.15.10
jobs:
goreleaser:


+ 5
- 5
.goreleaser.yml View File

@ -29,8 +29,8 @@ release:
archives:
- files:
- LICENSE
- README.md
- UPGRADING.md
- SECURITY.md
- CHANGELOG.md
- LICENSE
- README.md
- UPGRADING.md
- SECURITY.md
- CHANGELOG.md

+ 2
- 2
docker-compose.yml View File

@ -65,5 +65,5 @@ networks:
ipam:
driver: default
config:
-
subnet: 192.167.10.0/16
-
subnet: 192.167.10.0/16

+ 2
- 5
networks/remote/ansible/config.yml View File

@ -1,8 +1,6 @@
---
#Requires BINARY and CONFIGDIR variables set.
#N=4 hosts by default.
# Requires BINARY and CONFIGDIR variables set.
# N=4 hosts by default.
- hosts: all
user: root
any_errors_fatal: true
@ -15,4 +13,3 @@
- config
- unsafe_reset
- start

+ 0
- 2
networks/remote/ansible/install.yml View File

@ -1,5 +1,4 @@
---
- hosts: all
user: root
any_errors_fatal: true
@ -8,4 +7,3 @@
- service: tendermint
roles:
- install

+ 1
- 4
networks/remote/ansible/logzio.yml View File

@ -1,7 +1,5 @@
---
#Note: You need to add LOGZIO_TOKEN variable with your API key. Like tihs: ansible-playbook -e LOGZIO_TOKEN=ABCXYZ123456
# Note: You need to add LOGZIO_TOKEN variable with your API key. Like tihs: ansible-playbook -e LOGZIO_TOKEN=ABCXYZ123456
- hosts: all
user: root
any_errors_fatal: true
@ -11,4 +9,3 @@
- JOURNALBEAT_BINARY: "{{lookup('env', 'GOPATH')}}/bin/journalbeat"
roles:
- logzio

+ 0
- 3
networks/remote/ansible/reset.yml View File

@ -1,5 +1,4 @@
---
- hosts: all
user: root
any_errors_fatal: true
@ -10,5 +9,3 @@
- stop
- unsafe_reset
- start

+ 0
- 2
networks/remote/ansible/restart.yml View File

@ -1,5 +1,4 @@
---
- hosts: all
user: root
any_errors_fatal: true
@ -9,4 +8,3 @@
roles:
- stop
- start

+ 1
- 3
networks/remote/ansible/roles/config/tasks/main.yml View File

@ -1,5 +1,4 @@
---
- name: Copy binary
copy:
src: "{{BINARY}}"
@ -13,5 +12,4 @@
dest: "/home/{{service}}/.{{service}}/"
owner: "{{service}}"
group: "{{service}}"
loop: [ 0, 1, 2, 3, 4, 5, 6, 7 ]
loop: [0, 1, 2, 3, 4, 5, 6, 7]

+ 0
- 2
networks/remote/ansible/roles/install/handlers/main.yml View File

@ -1,5 +1,3 @@
---
- name: reload services
systemd: "name={{service}} daemon_reload=yes enabled=yes"

+ 0
- 2
networks/remote/ansible/roles/install/tasks/main.yml View File

@ -1,5 +1,4 @@
---
- name: Create service group
group: "name={{service}}"
@ -12,4 +11,3 @@
- name: Create service
template: "src=systemd.service.j2 dest=/etc/systemd/system/{{service}}.service"
notify: reload services

+ 0
- 2
networks/remote/ansible/roles/logzio/handlers/main.yml View File

@ -1,8 +1,6 @@
---
- name: reload daemon
command: "systemctl daemon-reload"
- name: restart journalbeat
service: name=journalbeat state=restarted

+ 0
- 2
networks/remote/ansible/roles/logzio/tasks/main.yml View File

@ -1,5 +1,4 @@
---
- name: Copy journalbeat binary
copy: src="{{JOURNALBEAT_BINARY}}" dest=/usr/bin/journalbeat mode=0755
notify: restart journalbeat
@ -24,4 +23,3 @@
notify:
- reload daemon
- restart journalbeat

+ 0
- 2
networks/remote/ansible/roles/start/tasks/main.yml View File

@ -1,5 +1,3 @@
---
- name: start service
service: "name={{service}} state=started"

+ 0
- 2
networks/remote/ansible/roles/status/tasks/main.yml View File

@ -1,5 +1,4 @@
---
- name: application service status
command: "service {{service}} status"
changed_when: false
@ -7,4 +6,3 @@
- name: Result
debug: var=status.stdout_lines

+ 0
- 2
networks/remote/ansible/roles/stop/tasks/main.yml View File

@ -1,5 +1,3 @@
---
- name: stop service
service: "name={{service}} state=stopped"

+ 0
- 1
networks/remote/ansible/roles/unsafe_reset/tasks/main.yml View File

@ -1,4 +1,3 @@
- command: "{{service}} unsafe_reset_all {{ (service != 'tendermint') | ternary('node','') }} --home /home/{{service}}/.{{service}}"
become_user: "{{service}}"
become: yes

+ 0
- 2
networks/remote/ansible/start.yml View File

@ -1,5 +1,4 @@
---
- hosts: all
user: root
any_errors_fatal: true
@ -8,4 +7,3 @@
- service: tendermint
roles:
- start

+ 0
- 2
networks/remote/ansible/status.yml View File

@ -1,5 +1,4 @@
---
- hosts: all
user: root
any_errors_fatal: true
@ -8,4 +7,3 @@
- service: tendermint
roles:
- status

+ 0
- 2
networks/remote/ansible/stop.yml View File

@ -1,5 +1,4 @@
---
- hosts: all
user: root
any_errors_fatal: true
@ -8,4 +7,3 @@
- service: tendermint
roles:
- stop

Loading…
Cancel
Save