search

stratum / fabric-tna

The SD-Fabric data plane
https://docs.sd-fabric.org/
30 stars 15 forks source link

Update license to Apache-2.0 #475

Closed ccascone closed 2 years ago

ccascone commented 2 years ago

In preparation for the open source release

codecov[bot] commented 2 years ago

Codecov Report

Merging #475 (1ed0174) into main (8891e3f) will not change coverage. The diff coverage is n/a.

:exclamation: Current head 1ed0174 differs from pull request most recent head 4418466. Consider uploading reports for the commit 4418466 to get more accurate results

Impacted file tree graph

@@            Coverage Diff            @@
##               main     #475   +/-   ##
=========================================
  Coverage     69.16%   69.16%           
  Complexity      696      696           
=========================================
  Files            60       60           
  Lines          4634     4634           
  Branches        514      514           
=========================================
  Hits           3205     3205           
  Misses         1168     1168           
  Partials        261      261
Impacted Files Coverage Δ
.../java/org/stratumproject/fabric/tna/Constants.java 100.00% <ø> (ø)
.../org/stratumproject/fabric/tna/PipeconfLoader.java 0.00% <ø> (ø)
...c/tna/behaviour/AbstractFabricHandlerBehavior.java 72.72% <ø> (ø)
...oject/fabric/tna/behaviour/FabricCapabilities.java 25.00% <ø> (ø)
...ct/fabric/tna/behaviour/FabricIntProgrammable.java 86.01% <ø> (ø)
...roject/fabric/tna/behaviour/FabricInterpreter.java 82.38% <ø> (ø)
...bric/tna/behaviour/FabricTreatmentInterpreter.java 67.74% <ø> (ø)
...ratumproject/fabric/tna/behaviour/FabricUtils.java 77.58% <ø> (ø)
...mproject/fabric/tna/behaviour/P4InfoConstants.java 100.00% <ø> (ø)
...haviour/pipeliner/AbstractObjectiveTranslator.java 88.23% <ø> (ø)
... and 50 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 8891e3f...4418466. Read the comment docs.

ccascone commented 2 years ago

@Yi-Tseng please take a look at https://github.com/stratum/fabric-tna/pull/475/commits/44184662b1d262ef029a37dd7625af1f209a01f8

That will require the following change to the script generating bmd_bytes: https://github.com/stratum/bf-sde-docker/pull/14/files

Currently CI is failing because the generated bmd_bytes.py has the member-only license.

Yi-Tseng commented 2 years ago

retest this please

Yi-Tseng commented 2 years ago

@Yi-Tseng please take a look at 4418466

That will require the following change to the script generating bmd_bytes: https://github.com/stratum/bf-sde-docker/pull/14/files

Currently CI is failing because the generated bmd_bytes.py has the member-only license.

Done, the p4c image is updated with your patch. testing now

Yi-Tseng commented 2 years ago

retest this please