8000 fix(ci): resolving cpplint issues by athiravinod5gtechie · Pull Request #1 · athiravinod5gtechie/magma · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

fix(ci): resolving cpplint issues #1

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

Draft
wants to merge 131 commits into
base: master
Choose a base branch
from

Conversation

athiravinod5gtechie
Copy link
Owner

Summary

Test Plan

Additional Information

  • This change is backwards-breaking

Security Considerations

dependabot bot and others added 30 commits June 1, 2023 14:43
)

Bumps [github.com/docker/distribution](https://github.com/docker/distribution) from 2.8.1+incompatible to 2.8.2+incompatible.
- [Release notes](https://github.com/docker/distribution/releases)
- [Commits](distribution/distribution@v2.8.1...v2.8.2)

---
updated-dependencies:
- dependency-name: github.com/docker/distribution
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Force merge with go from Lucas Gonze: https://magmacore.slack.com/archives/C01Q1T14YJ2/p1687184855726759

* fix(nms): nginx ssl issue

Signed-off-by: prabina pattnaik <prabinak@wavelabs.ai>

* fix(nms): nginx ssl issue

Signed-off-by: prabina pattnaik <prabinak@wavelabs.ai>

* fix(nms): nginx ssl issue

Signed-off-by: prabina pattnaik <prabinak@wavelabs.ai>

---------

Signed-off-by: prabina pattnaik <prabinak@wavelabs.ai>
Force merge, with go and review from @lucasgonze.

* fix(ci): ci dashboard issue

Signed-off-by: moyilakhila <akhila.moyila@wavelabs.ai>

* fix(ci): ci trigger for pull request

Signed-off-by: moyilakhila <akhila.moyila@wavelabs.ai>

---------

Signed-off-by: moyilakhila <akhila.moyila@wavelabs.ai>
Co-authored-by: moyilakhila <akhila.moyila@wavelabs.ai>
force merged with review by @panyogesh 

Signed-off-by: Maximilian Huber <maximilian.huber@tngtech.com>
…5224)

Force merge to work on unblocking CI.

the proposed solution is taken from
thisbejim/Pyrebase#406

Signed-off-by: Maximilian Huber <maximilian.huber@tngtech.com>
…meta data (#15158)" (#15226)

Force merge to work on unblocking CI.

This reverts commit 38c74d1.

Signed-off-by: Maximilian Huber <maximilian.huber@tngtech.com>
force merge, to unblock CI

Change:
  1. In build_all.yml inherting the secrets.

Signed-off-by: moyilakhila <akhila.moyila@wavelabs.ai>
Co-authored-by: moyilakhila <akhila.moyila@wavelabs.ai>
Changes:
 1. Maximize the build space

Signed-off-by: moyilakhila <akhila.moyila@wavelabs.ai>
Co-authored-by: moyilakhila <akhila.moyila@wavelabs.ai>
…on (#15001)

The issue #14995 is caused by a missing folder `alert_rules` in the prometheus configuration folder. The `alertmanager-configurer` will create the folder with restrictive access rights which will prevent `prometheus` and `alertmanager` to read the configuration.

Signed-off-by: Jordan Vrtanoski <jordan.vrtanoski@gmail.com>
Changes:
  1. Added testcases for big nas
  2. Add check for big packets

Test:
  - Added new unit testcase

Signed-off-by: Yogesh Pandey <yogesh@wavelabs.ai>
Changes:
  1. Handling of RAN_UE_ID missing

Testing:
  1. Adding unit testcase for missing ran ue id.

Signed-off-by: Yogesh Pandey <yogesh@wavelabs.ai>
The ulr command allows the s6a_cli to issue an Update Location Request
(ULR) to an HSS, and parse the received Update Location Answer (ULA).


Reviewed-by: Roger Pueyo Centelles <roger.pueyo@i2cat.net>

Signed-off-by: Miguel Fuentes <miguel.fuentes@i2cat.net>
Co-authored-by: Miguel Fuentes <miguel.fuentes@i2cat.net>
Signed-off-by: mehul-jindal-wavelabs <mehul.jindal@wavelabs.ai>
…yment (#14863)

Signed-off-by: Jim Blakley <jblake1@andrew.cmu.edu>
Signed-off-by: krishnavamsi-wavelabs <krishna.vamsi@wavelabs.ai>
* Fix #15293 Add Rewards Program Documentation

Signed-off-by: Benjamin Sternthal <ben@devpatch.com>

* fixed lint issues

Signed-off-by: Benjamin Sternthal <ben@devpatch.com>

---------

Signed-off-by: Benjamin Sternthal <ben@devpatch.com>
An Update Location Answer (ULA) may contain, inside the APN
configuration, an IPv4 address and an IPv6 address assigned by the HSS
(i.e., the Served-Party-IP-Address AVP, which is optional). The
s6a_proxy at the FeG would receive the Served-Party-IP-Address AVP in
the ULA from an HSS, as visible in the logs, but would not process it
when sending the data in protobuffer (e.g., to the AGW).

This patch adds the missing lines to handle the Served-Party-IP-Address
AVP in a ULA, and include it in the protobuffer representation of the
APN it belongs to.

Signed-off-by: Roger Pueyo Centelles <roger.pueyo@i2cat.net>
Signed-off-by: krishnavamsi-wavelabs <krishna.vamsi@wavelabs.ai>
…1AP_ID (#15309)

Signed-off-by: krishnavamsi-wavelabs <krishna.vamsi@wavelabs.ai>
Temporary fixes to update the bazel-base image.
…mage (#15323)

* fix(ci): Fixes for liblsan0 packages in devcontainer and bazel base image

changes:
 1) Added the workflow_dispatch condition

Signed-off-by: akhilamoyila9 <akhila.moyila@wavelabs.ai>

* fix(ci): Fixes for liblsan0 packages in devcontainer and bazel base image

changes:
 1. Modified the workflow dispatch condition

Signed-off-by: akhilamoyila9 <akhila.moyila@wavelabs.ai>

* fix(ci): Fixes for liblsan0 packages in devcontainer and bazel base image

changes:
 1. Removed the white spaces in the workflow file

Signed-off-by: akhilamoyila9 <akhila.moyila@wavelabs.ai>

* fix(ci): Fixes for liblsan0 packages in devcontainer and bazel base image

changes:
 1) Modified the github event conditions

Signed-off-by: akhilamoyila9 <akhila.moyila@wavelabs.ai>

---------

Signed-off-by: akhilamoyila9 <akhila.moyila@wavelabs.ai>
changes:
 1) modified the .bazelrc file
 2) removed flags in the production build

Signed-off-by: mohinali <mohinali989@gmail.com>
Co-authored-by: mohinali <mohinali989@gmail.com>
* Whitelist Net type to prevent web inject attack

Signed-off-by: jahid-wavelabs <jahidul.mallick@wavelabs.ai>

* Whitelist N/W type to prevent web inject attack

Signed-off-by: jahid-wavelabs <jahidul.mallick@wavelabs.ai>

* Whitelist N/W type to prevent web inject attack

Signed-off-by: jahid-wavelabs <jahidul.mallick@wavelabs.ai>

---------

Signed-off-by: jahid-wavelabs <jahidul.mallick@wavelabs.ai>
Co-authored-by: jahid-wavelabs <jahidul.mallick@wavelabs.ai>
changes:
 1) Modified the PUSH_TO_REGISTRY condition for bazel_base and devcontainer
 2) Added workflow dispatch for bazel_cache_asan,bazel_cache_plain and bazel_cache_prod

Signed-off-by: akhilamoyila9 <akhila.moyila@wavelabs.ai>
…th uplink data status (#15330)

Signed-off-by: Akshayp77 <akshay.patidar@veltris.com>
Co-authored-by: panyogesh <69527565+panyogesh@users.noreply.github.com>
… PathSwitchRequestAcknowlidge (#15337)

Signed-off-by: Sathyaj27 <sathya.jayadev@veltris.com>
Signed-off-by: krishnavamsi-wavelabs <krishna.vamsi@wavelabs.ai>
Co-authored-by: panyogesh <69527565+panyogesh@users.noreply.github.com>
…_ID in E-RAB Modification Indication (#15326)

Signed-off-by: krishnavamsi-wavelabs <krishna.vamsi@wavelabs.ai>
Co-authored-by: panyogesh <69527565+panyogesh@users.noreply.github.com>
Signed-off-by: mehul-jindal <mehul.jindal@veltris.com>
* fix(feg): replace golang package repository

Signed-off-by: mehul-jindal <mehul.jindal@veltris.com>

* fix(workflows): Allow GO v1.21.0 through Golang version check for Dockerfiles

Signed-off-by: mehul-jindal <mehul.jindal@veltris.com>

---------

Signed-off-by: mehul-jindal <mehul.jindal@veltris.com>
dependabot bot and others added 10 commits March 14, 2025 16:56
Bumps [dompurify](https://github.com/cure53/DOMPurify) from 2.3.6 to 2.5.6.
- [Release notes](https://github.com/cure53/DOMPurify/releases)
- [Commits](cure53/DOMPurify@2.3.6...2.5.6)

---
updated-dependencies:
- dependency-name: dompurify
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [express](https://github.com/expressjs/express) from 4.17.3 to 4.20.0.
- [Release notes](https://github.com/expressjs/express/releases)
- [Changelog](https://github.com/expressjs/express/blob/master/History.md)
- [Commits](expressjs/express@4.17.3...4.20.0)

---
updated-dependencies:
- dependency-name: express
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.53.0 to 1.56.3.
- [Release notes](https://github.com/grpc/grpc-go/releases)
- [Commits](grpc/grpc-go@v1.53.0...v1.56.3)

---
updated-dependencies:
- dependency-name: google.golang.org/grpc
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…13831

Introducing new conditional code that will be activated when compiled on
earlier kernels.
The conditional code will use the older version of the API
`ipv6_stub->ipv6_dst_lookup` instead of the newer one
`ipv6_stub->ipv6_dst_lookup_flow`. The code will not be active on
kernels that have implementation of `ipv6_stub->ipv6_dst_lookup_flow()`

Signed-off-by: Jordan Vrtanoski <jordan.vrtanoski@gmail.com>
* feat(agw): Basic eDRX support

Signed-off-by: Gustavo Junior Alves <gjalves@gjalves.com.br>

* feat(agw): Basic eDRX support

Signed-off-by: Gustavo Junior Alves <gjalves@gjalves.com.br>

* feat(agw): Basic eDRX support

Signed-off-by: Gustavo Junior Alves <gjalves@gjalves.com.br>

* feat(agw): Basic eDRX support

Signed-off-by: Gustavo Junior Alves <gjalves@gjalves.com.br>

---------

Signed-off-by: Gustavo Junior Alves <gjalves@gjalves.com.br>
Bumps [cryptography](https://github.com/pyca/cryptography) from 41.0.3 to 43.0.1.
- [Changelog](https://github.com/pyca/cryptography/blob/main/CHANGELOG.rst)
- [Commits](pyca/cryptography@41.0.3...43.0.1)

---
updated-dependencies:
- dependency-name: cryptography
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [golang.org/x/net](https://github.com/golang/net) from 0.7.0 to 0.36.0.
- [Commits](golang/net@v0.7.0...v0.36.0)

---
updated-dependencies:
- dependency-name: golang.org/x/net
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: Gabryel Nóbrega <gabryelster@gmail.com>
Signed-off-by: Lucas Gonze <lucas@gonze.com>
Co-authored-by: Jordan Vrtanoski <1470453+jordanvrtanoski@users.noreply.github.com>
The key pair used for the Artifactory registry has been changed, and
needs to be updated here to fix CI builds.

Signed-off-by: Eric Ball <eball@linuxfoundation.org>
dependabot bot and others added 16 commits March 19, 2025 09:17
Bumps [canvg](https://github.com/canvg/canvg) from 3.0.10 to 3.0.11.
- [Release notes](https://github.com/canvg/canvg/releases)
- [Changelog](https://github.com/canvg/canvg/blob/v3.0.11/CHANGELOG.md)
- [Commits](https://github.com/canvg/canvg/commits/v3.0.11)

---
updated-dependencies:
- dependency-name: canvg
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [@babel/runtime](https://github.com/babel/babel/tree/HEAD/packages/babel-runtime) from 7.17.8 to 7.26.10.
- [Release notes](https://github.com/babel/babel/releases)
- [Changelog](https://github.com/babel/babel/blob/main/CHANGELOG.md)
- [Commits](https://github.com/babel/babel/commits/v7.26.10/packages/babel-runtime)

---
updated-dependencies:
- dependency-name: "@babel/runtime"
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [@babel/helpers](https://github.com/babel/babel/tree/HEAD/packages/babel-helpers) from 7.14.0 to 7.26.10.
- [Release notes](https://github.com/babel/babel/releases)
- [Changelog](https://github.com/babel/babel/blob/main/CHANGELOG.md)
- [Commits](https://github.com/babel/babel/commits/v7.26.10/packages/babel-helpers)

---
updated-dependencies:
- dependency-name: "@babel/helpers"
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [xml-crypto](https://github.com/node-saml/xml-crypto) from 2.1.3 to 2.1.6.
- [Release notes](https://github.com/node-saml/xml-crypto/releases)
- [Changelog](https://github.com/node-saml/xml-crypto/blob/master/CHANGELOG.md)
- [Commits](node-saml/xml-crypto@v2.1.3...v2.1.6)

---
updated-dependencies:
- dependency-name: xml-crypto
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [nanoid](https://github.com/ai/nanoid) from 3.3.4 to 3.3.8.
- [Release notes](https://github.com/ai/nanoid/releases)
- [Changelog](https://github.com/ai/nanoid/blob/main/CHANGELOG.md)
- [Commits](ai/nanoid@3.3.4...3.3.8)

---
updated-dependencies:
- dependency-name: nanoid
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [cryptography](https://github.com/pyca/cryptography) from 43.0.1 to 44.0.1.
- [Changelog](https://github.com/pyca/cryptography/blob/main/CHANGELOG.rst)
- [Commits](pyca/cryptography@43.0.1...44.0.1)

---
updated-dependencies:
- dependency-name: cryptography
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…#15486)

Bumps [requests](https://github.com/psf/requests) from 2.31.0 to 2.32.2.
- [Release notes](https://github.com/psf/requests/releases)
- [Changelog](https://github.com/psf/requests/blob/main/HISTORY.md)
- [Commits](psf/requests@v2.31.0...v2.32.2)

---
updated-dependencies:
- dependency-name: requests
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…#15637)

* fix(nms): Code scanning alert no. 141: DOM text reinterpreted as HTML

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Signed-off-by: Jordan Vrtanoski <jordan.vrtanoski@gmail.com>

* Update nms/app/components/admin/Networks.tsx

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Signed-off-by: Jordan Vrtanoski <jordan.vrtanoski@gmail.com>

---------

Signed-off-by: Jordan Vrtanoski <jordan.vrtanoski@gmail.com>
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Bumps [golang.org/x/crypto](https://github.com/golang/crypto) from 0.24.0 to 0.31.0.
- [Commits](golang/crypto@v0.24.0...v0.31.0)

---
updated-dependencies:
- dependency-name: golang.org/x/crypto
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
The runner is pulling the git repo with "runner" user while
the process runds in docker container with "root" as user
which causes issue with the ownership of the local repo.

The fix is instructing git tool to ignore this error

Signed-off-by: Jordan Vrtanoski <jordan.vrtanoski@gmail.com>
#15401)

Signed-off-by: Nathaniel <me@nathanielbennett.com>
#15538 because I don't have write to that and it needs a little work to go through. Also fix clang-lint. (#15560)

Signed-off-by: Lucas Gonze <lucas@gonze.com>
…15528)

Bumps [gitpython](https://github.com/gitpython-developers/GitPython) from 3.1.32 to 3.1.41.
- [Release notes](https://github.com/gitpython-developers/GitPython/releases)
- [Changelog](https://github.com/gitpython-developers/GitPython/blob/main/CHANGES)
- [Commits](gitpython-developers/GitPython@3.1.32...3.1.41)

---
updated-dependencies:
- dependency-name: gitpython
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* fix(agw): upgrade asn tool with latest

Signed-off-by: Ganesh Gedela <ganesh.gedela@veltris.com>

* updated docker files based new asn repo

Signed-off-by: Ganesh Gedela <ganesh.gedela@veltris.com>

---------

Signed-off-by: Ganesh Gedela <ganesh.gedela@veltris.com>
Bumps [golang.org/x/crypto](https://github.com/golang/crypto) from 0.0.0-20220722155217-630584e8d5aa to 0.31.0.
- [Commits](https://github.com/golang/crypto/commits/v0.31.0)

---
updated-dependencies:
- dependency-name: golang.org/x/crypto
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.c
DBC1
om>
)

- Update M5GSUCIRegistrationServiceClient.cpp for proper SUCI handling
- Modify nas_proc.cpp to fix MSIN decoding process
- Adjust test_amf_procedures.cpp for debugging

Signed-off-by: kmu-tm <kamar.mubasier@telenormaritime.com>
jordanvrtanoski and others added 2 commits March 31, 2025 16:42
Eliminated potential weakness from hardcoded session token.

Signed-off-by: Jordan Vrtanoski <jordan.vrtanoski@gmail.com>
* fix(nms): Upload and Download data switched and misaligned

Signed-off-by: LucasDamascenoS <lucasdamasceno484@gmail.com>

* fix(nms): data plan bitrate information inverted

Signed-off-by: LucasDamascenoS <lucasdamasceno484@gmail.com>

---------

Signed-off-by: LucasDamascenoS <lucasdamasceno484@gmail.com>
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.

0