integ

starlingx/integ

Commit Graph

Author	SHA1	Message	Date
Leonardo Fagundes Luz Serrano	5e19b07ebb	tox: fixed warnings meta_data files: - "too few spaces before comment" warning - missing document start "---" warning tox.ini: - allowlist bash for testenv - moved 'basepython' var to testenv as it was repeated in all derived envs - removed "tox < 4" requirement. changed minversion to 4 pylint: - renamed pylintrc to the standard convention ".pylintrc" - Removed "enable" section, as all warnings are enabled by default - Updated "disable" section as some warnings changed names - files-output deprecated [1] - bad-functions requires a plugin [2, 3] - max-branchs ==> max-branches [4] - overgeneral-exceptions updated [1] https://pylint.pycqa.org/en/latest/whatsnew/1/1.7/summary.html#removed-changes [2] https://pylint.pycqa.org/en/latest/user_guide/configuration/all-options.html#bad-functions [3] https://pylint.pycqa.org/en/latest/user_guide/messages/warning/bad-builtin.html [4] https://pylint.readthedocs.io/en/latest/user_guide/messages/refactor/too-many-branches.html Test Plan: pass: run tox Partial-Bug: 1976377 Change-Id: I715ed859e046a90cc7ec5d047d42902a51009eb8 Signed-off-by: Leonardo Fagundes Luz Serrano <Leonardo.FagundesLuzSerrano@windriver.com>	2023-09-06 17:54:55 -03:00
Manoel Benedito Neto	c15e3e1a3a	Fix lint errors identified by Zuul pylint job This commit fixes lint errors identified by Zuul after stx-integ-pylint job is executed. Test Plan: PASS: stx-integ-pylint job is executed successfully. PASS: Run "yamllint ." command on integ repo base directory. Observe that no lint errors of line-length, truthy, indentation, new-line-at-end-of-file and document-start are listed. PASS: build-pkgs -a -c Closes-Bug: 2011632 Change-Id: I4d8229b5de8c9d88ff2aab6169521ab377b5866c Signed-off-by: Manoel Benedito Neto <manoel.beneditoneto@windriver.com>	2023-03-15 12:07:17 +00:00
Steven Webster	75e3878176	debian-pkg: Uprev containernetworking-plugins to 1.0.1 This commit uprevs the containernetworking-plugins to 1.0.1 on Debian. The version of this package has a Build-Depends on: golang-github-appc-cni-dev >= 1.0.1 golang-github-vishvananda-netlink-dev >= 1.1.0.125 Currently, the Debian Bullseye release provides: golang-github-appc-cni-dev = 0.8.1 golang-github-vishvananda-netlink-dev = 1.1.0-2 So in order to build the containernetworking-plugins 1.0.1, this commit pulls in the appropriate versions of the dependencies and builds them too. The other thing to note in this commit is that the plugin install path has been changed from /usr/lib/cni/ to /usr/libexec/cni/. This aligns with where k8s expects to find CNI plugins on StarlingX Testing: - PASS: downloader -s - PASS: full build - PASS: install and ensure the plugins are present at /usr/libexec/cni/ - PASS: install and ensure the package is the correct version Story: 2009832 Task: 44635 Signed-off-by: Steven Webster <steven.webster@windriver.com> Change-Id: Ic1c9a0d7eb2adb831c6316e1ab72d288ac805929	2022-03-18 09:53:07 -04:00

Author

SHA1

Message

Date

Leonardo Fagundes Luz Serrano

5e19b07ebb

tox: fixed warnings

meta_data files:
- "too few spaces before comment" warning
- missing document start "---" warning

tox.ini:
- allowlist bash for testenv
- moved 'basepython' var to testenv as it was repeated
  in all derived envs
- removed "tox < 4" requirement. changed minversion to 4

pylint:
- renamed pylintrc to the standard convention ".pylintrc"
- Removed "enable" section, as all warnings are enabled by default
- Updated "disable" section as some warnings changed names
- files-output deprecated [1]
- bad-functions requires a plugin [2, 3]
- max-branchs ==> max-branches [4]
- overgeneral-exceptions updated

[1] https://pylint.pycqa.org/en/latest/whatsnew/1/1.7/summary.html#removed-changes
[2] https://pylint.pycqa.org/en/latest/user_guide/configuration/all-options.html#bad-functions
[3] https://pylint.pycqa.org/en/latest/user_guide/messages/warning/bad-builtin.html
[4] https://pylint.readthedocs.io/en/latest/user_guide/messages/refactor/too-many-branches.html

Test Plan:
pass: run tox

Partial-Bug: 1976377

Change-Id: I715ed859e046a90cc7ec5d047d42902a51009eb8
Signed-off-by: Leonardo Fagundes Luz Serrano <Leonardo.FagundesLuzSerrano@windriver.com>

2023-09-06 17:54:55 -03:00

Manoel Benedito Neto

c15e3e1a3a

Fix lint errors identified by Zuul pylint job

This commit fixes lint errors identified by Zuul after stx-integ-pylint
job is executed.

Test Plan:
PASS: stx-integ-pylint job is executed successfully.
PASS: Run "yamllint ." command on integ repo base directory. Observe
      that no lint errors of line-length, truthy, indentation,
      new-line-at-end-of-file and document-start are listed.
PASS: build-pkgs -a -c

Closes-Bug: 2011632
Change-Id: I4d8229b5de8c9d88ff2aab6169521ab377b5866c
Signed-off-by: Manoel Benedito Neto <manoel.beneditoneto@windriver.com>

2023-03-15 12:07:17 +00:00

Steven Webster

75e3878176

debian-pkg: Uprev containernetworking-plugins to 1.0.1

This commit uprevs the containernetworking-plugins to 1.0.1 on
Debian.

The version of this package has a Build-Depends on:

golang-github-appc-cni-dev >= 1.0.1
golang-github-vishvananda-netlink-dev >= 1.1.0.125

Currently, the Debian Bullseye release provides:

golang-github-appc-cni-dev = 0.8.1
golang-github-vishvananda-netlink-dev = 1.1.0-2

So in order to build the containernetworking-plugins 1.0.1, this
commit pulls in the appropriate versions of the dependencies
and builds them too.

The other thing to note in this commit is that the plugin install
path has been changed from /usr/lib/cni/ to /usr/libexec/cni/.
This aligns with where k8s expects to find CNI plugins on StarlingX

Testing:

- PASS: downloader -s
- PASS: full build
- PASS: install and ensure the plugins are present
        at /usr/libexec/cni/
- PASS: install and ensure the package is the correct version

Story: 2009832
Task: 44635

Signed-off-by: Steven Webster <steven.webster@windriver.com>
Change-Id: Ic1c9a0d7eb2adb831c6316e1ab72d288ac805929

2022-03-18 09:53:07 -04:00

3 Commits