Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refine p4cp github lint workflows #398

Merged
merged 1 commit into from
Jan 30, 2024
Merged

Refine p4cp github lint workflows #398

merged 1 commit into from
Jan 30, 2024

Conversation

ffoulkes
Copy link
Contributor

@ffoulkes ffoulkes commented Jan 28, 2024

  • Rename lintdocs.yml to linters.yml.

  • Move py_bandit_check and clang_format_check to linters.yml.

  • Expand py_bandit_check to apply to all .py files.

@ffoulkes ffoulkes force-pushed the more-bandit branch 4 times, most recently from ea44824 to 7282fe3 Compare January 28, 2024 16:27
@ffoulkes ffoulkes changed the title Update bandit github workflow Refine p4cp github lint workflows Jan 28, 2024
@ffoulkes ffoulkes marked this pull request as ready for review January 28, 2024 16:32
@ffoulkes ffoulkes added minor effort Minimal effort required github_actions Pull requests that update GitHub Actions code labels Jan 28, 2024
@ffoulkes ffoulkes force-pushed the more-bandit branch 3 times, most recently from ea607de to 3f9cc4b Compare January 28, 2024 16:52
Copy link
Collaborator

@5abeel 5abeel left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Contributor

@nupuruttarwar nupuruttarwar left a comment

Choose a reason for hiding this comment

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

LGTM

@nupuruttarwar
Copy link
Contributor

Please resolve the conflict before merging

- Rename lintdocs.yml to linters.yml.

- Move py_bandit_check and clang_format_check to linters.yml.

- Expand py_bandit_check to apply to all .py files.

Signed-off-by: Derek G Foster <derek.foster@intel.com>
@ffoulkes ffoulkes merged commit 92dc661 into main Jan 30, 2024
9 checks passed
@ffoulkes ffoulkes deleted the more-bandit branch January 30, 2024 01:53
satish153 pushed a commit that referenced this pull request Feb 1, 2024
- Rename lintdocs.yml to linters.yml.

- Move py_bandit_check and clang_format_check to linters.yml.

- Expand py_bandit_check to apply to all .py files.

Signed-off-by: Derek G Foster <derek.foster@intel.com>
Signed-off-by: Satish Pitchikala <satish.pitchikala@intel.com>
satish153 pushed a commit that referenced this pull request Feb 2, 2024
- Rename lintdocs.yml to linters.yml.

- Move py_bandit_check and clang_format_check to linters.yml.

- Expand py_bandit_check to apply to all .py files.

Signed-off-by: Derek G Foster <derek.foster@intel.com>
Signed-off-by: Satish Pitchikala <satish.pitchikala@intel.com>
satish153 added a commit that referenced this pull request Feb 9, 2024
* geneve support in LNW.p4

Signed-off-by: Satish Pitchikala <satish.pitchikala@intel.com>

* added actions for geneve

Signed-off-by: Satish Pitchikala <satish.pitchikala@intel.com>

* geneve dcap actions

Signed-off-by: Satish Pitchikala <satish.pitchikala@intel.com>

* fix for dpdk target build fail

Signed-off-by: Satish Pitchikala <satish.pitchikala@intel.com>

* fix for issue while deleting entries

Signed-off-by: Satish Pitchikala <satish.pitchikala@intel.com>

* updated ovs refpoint

Signed-off-by: Satish Pitchikala <satish.pitchikala@intel.com>

* Update ovs-p4rt/ovs_p4rt.cc

Co-authored-by: Derek G Foster <derek.foster@intel.com>
Signed-off-by: Satish Pitchikala <satish.pitchikala@intel.com>

* Update ovs-p4rt/ovs_p4rt.cc

Co-authored-by: Derek G Foster <derek.foster@intel.com>
Signed-off-by: Satish Pitchikala <satish.pitchikala@intel.com>

* Update ovs-p4rt/ovs_p4rt.cc

Co-authored-by: Derek G Foster <derek.foster@intel.com>
Signed-off-by: Satish Pitchikala <satish.pitchikala@intel.com>

* Update ovs-p4rt/ovs_p4rt.cc

Co-authored-by: Derek G Foster <derek.foster@intel.com>
Signed-off-by: Satish Pitchikala <satish.pitchikala@intel.com>

* Bump robinraju/release-downloader from 1.8 to 1.9 (#400)

Bumps [robinraju/release-downloader](https://github.com/robinraju/release-downloader) from 1.8 to 1.9.
- [Release notes](https://github.com/robinraju/release-downloader/releases)
- [Commits](robinraju/release-downloader@v1.8...v1.9)

---
updated-dependencies:
- dependency-name: robinraju/release-downloader
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: Satish Pitchikala <satish.pitchikala@intel.com>

* Bump tj-actions/changed-files from 41 to 42 (#399)

Bumps [tj-actions/changed-files](https://github.com/tj-actions/changed-files) from 41 to 42.
- [Release notes](https://github.com/tj-actions/changed-files/releases)
- [Changelog](https://github.com/tj-actions/changed-files/blob/main/HISTORY.md)
- [Commits](tj-actions/changed-files@v41...v42)

---
updated-dependencies:
- dependency-name: tj-actions/changed-files
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: Satish Pitchikala <satish.pitchikala@intel.com>

* Refine p4cp github lint workflows (#398)

- Rename lintdocs.yml to linters.yml.

- Move py_bandit_check and clang_format_check to linters.yml.

- Expand py_bandit_check to apply to all .py files.

Signed-off-by: Derek G Foster <derek.foster@intel.com>
Signed-off-by: Satish Pitchikala <satish.pitchikala@intel.com>

* P4CP : DefaultActionSupport (#397)

* P4Runtime Client Changes

Signed-off-by: Kumar, Aashish <aashish.kumar@intel.com>

* Addressing comments

Signed-off-by: Kumar, Aashish <aashish.kumar@intel.com>

* Addressing comments

Signed-off-by: Kumar, Aashish <aashish.kumar@intel.com>

---------

Signed-off-by: Kumar, Aashish <aashish.kumar@intel.com>
Signed-off-by: Satish Pitchikala <satish.pitchikala@intel.com>

* Update krnlmon submodule reference (#401)

Signed-off-by: Kumar, Aashish <aashish.kumar@intel.com>
Signed-off-by: Satish Pitchikala <satish.pitchikala@intel.com>

* clang format the code

Signed-off-by: Satish Pitchikala <satish.pitchikala@intel.com>

* fix for ipv6 tunnel port

Signed-off-by: Satish Pitchikala <satish.pitchikala@intel.com>

* Addressing review comments

Signed-off-by: Satish Pitchikala <satish.pitchikala@intel.com>

---------

Signed-off-by: Satish Pitchikala <satish.pitchikala@intel.com>
Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: Derek G Foster <derek.foster@intel.com>
Signed-off-by: Kumar, Aashish <aashish.kumar@intel.com>
Co-authored-by: Derek G Foster <derek.foster@intel.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: aashishkuma <aashish.kumar@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
github_actions Pull requests that update GitHub Actions code minor effort Minimal effort required
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants