Implementing graphql apis for label application rules

[kamaleshnneerasa]

Description

Please include a summary of the change, motivation and context.

Testing

Please describe the tests that you ran to verify your changes. Please summarize what did you test and what needs to be tested e.g. deployed and tested helm chart locally.

Checklist:

• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• Any dependent changes have been merged and published in downstream modules

Documentation

Make sure that you have documented corresponding changes in this repository or hypertrace docs repo if required.

[codecov]

Codecov Report

Merging #108 (47be40c) into main (ed9fa50) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##               main     #108      +/-   ##
============================================
+ Coverage     22.98%   23.03%   +0.04%     
  Complexity       75       75              
============================================
  Files            65       65              
  Lines          1679     1680       +1     
  Branches         52       52              
============================================
+ Hits            386      387       +1     
  Misses         1284     1284              
  Partials          9        9              

Flag	Coverage Δ
unit	23.03% <100.00%> (+0.04%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
...ava/org/hypertrace/graphql/impl/GraphQlModule.java	100.00% <100.00%> (ø)

---

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 ed9fa50...47be40c. Read the comment docs.

[aaron-steinfeld]

looks like not quite ready, but added early feedback.

[aaron-steinfeld]

Conflicts also need to be resolved

[aaron-steinfeld]

Let's fix snyk before merging please.

[aaron-steinfeld]

Let's fix snyk before merging please.

The vulnerability being raised is in gson, which comes in transitively. Adding a constraint to the platform similar to below should do the trick.

diff --git a/hypertrace-graphql-platform/build.gradle.kts b/hypertrace-graphql-platform/build.gradle.kts
index 0611e17..466c56e 100644
--- a/hypertrace-graphql-platform/build.gradle.kts
+++ b/hypertrace-graphql-platform/build.gradle.kts
@@ -13,5 +13,8 @@ dependencies {
     api("org.hypertrace.config.service:spaces-config-service-api:0.1.1")
     api("org.hypertrace.config.service:labels-config-service-api:0.1.8")
     api("org.hypertrace.config.service:label-application-rule-config-service-api:0.1.14")
+    api("com.google.code.gson:gson:2.8.9") {
+      because("https://snyk.io/vuln/SNYK-JAVA-COMGOOGLECODEGSON-1730327")
+    }
   }
 }
\ No newline at end of file

[github-actions]

Unit Test Results

11 files  ±0  11 suites  ±0   15s ⏱️ -3s
24 tests ±0  24 ✔️ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 597ad5f. ± Comparison against base commit ed9fa50.