forked from magma/magma
-
Notifications
You must be signed in to change notification settings - Fork 0
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
athiravinod5gtechie
wants to merge
131
commits into
athiravinod5gtechie:master
Choose a base branch
from
magma:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
) 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>
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>
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>
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>
…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>
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
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Test Plan
Additional Information
Security Considerations