operator-controller icon indicating copy to clipboard operation
operator-controller copied to clipboard

Catd graphql play

Open grokspawn opened this issue 4 months ago • 5 comments

Description

Reviewer Checklist

  • [ ] API Go Documentation
  • [ ] Tests: Unit Tests (and E2E Tests, if appropriate)
  • [ ] Comprehensive Commit Messages
  • [ ] Links to related GitHub Issue(s)

grokspawn avatar Jul 15 '25 16:07 grokspawn

Deploy Preview for olmv1 ready!

Name Link
Latest commit 346cf0d7fdbdfa499d04fec77fcadfa9df66d4d4
Latest deploy log https://app.netlify.com/projects/olmv1/deploys/691b25af266fb700089f22a8
Deploy Preview https://deploy-preview-2100--olmv1.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

netlify[bot] avatar Jul 15 '25 16:07 netlify[bot]

Codecov Report

:x: Patch coverage is 30.25641% with 408 lines in your changes missing coverage. Please review. :white_check_mark: Project coverage is 71.24%. Comparing base (c06f27f) to head (58fbad3).

Files with missing lines Patch % Lines
internal/catalogd/graphql/graphql.go 32.47% 245 Missing and 17 partials :warning:
internal/catalogd/server/handlers.go 21.69% 80 Missing and 3 partials :warning:
internal/catalogd/service/graphql_service.go 14.00% 43 Missing :warning:
internal/catalogd/server/http_helpers.go 0.00% 12 Missing :warning:
internal/catalogd/storage/localdir.go 66.66% 7 Missing and 1 partial :warning:
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2100      +/-   ##
==========================================
- Coverage   74.30%   71.24%   -3.07%     
==========================================
  Files          91       94       +3     
  Lines        7083     7476     +393     
==========================================
+ Hits         5263     5326      +63     
- Misses       1405     1719     +314     
- Partials      415      431      +16     
Flag Coverage Δ
e2e 43.50% <7.86%> (-2.15%) :arrow_down:
experimental-e2e 45.99% <8.20%> (-2.41%) :arrow_down:
unit 55.97% <24.27%> (-2.63%) :arrow_down:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

codecov[bot] avatar Jul 15 '25 16:07 codecov[bot]

closing this as stale - please reopen if needed

perdasilva avatar Sep 17 '25 15:09 perdasilva

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Once this PR has been reviewed and has the lgtm label, please assign tmshort for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

openshift-ci[bot] avatar Oct 28 '25 20:10 openshift-ci[bot]

PR needs rebase.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

openshift-merge-robot avatar Nov 20 '25 23:11 openshift-merge-robot