Skip to content
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

chore: fix package manifests to obtain Made for UDS "Silver" #1169

Open
Tracked by #727
justinthelaw opened this issue Sep 30, 2024 · 1 comment
Open
Tracked by #727

chore: fix package manifests to obtain Made for UDS "Silver" #1169

justinthelaw opened this issue Sep 30, 2024 · 1 comment
Assignees
Labels

Comments

@justinthelaw
Copy link
Contributor

Describe what should be investigated or refactored

Fixes need to be made based on the output of the Made for UDS "Silver" badge verification test. See the example failed test output below in additional context, and see the links to relevant code for details.

Links to any relevant code

See the tasks.yaml and the docs/DEVELOPMENT.md for details on how to run the tests.

See UDS Common's package practices for details on the "Must" requirements for a silver Made for UDS badge.

Additional context

Current (30 Sept, 2024) CPU verification test output:

-----------------------------
Package: leapfrogai-api

❌ Errors: 4
⚠️  Warnings: 3

❌ Error Descriptions:
  - Endpoint leapfrogai-api.uds.dev is returning 404
  - Not all applicable network policies are using selectors
  - Not all applicable network policies are using ports
  - No monitors defined

⚠️  Warning Descriptions:
  - Version is not consistent across flavors and package
  - Network policies with 'remoteGenerated: Anywhere' are present, review needed
  - No SSO configuration found, review needed
-----------------------------
Package: leapfrogai-ui

❌ Errors: 3
⚠️  Warnings: 4

❌ Error Descriptions:
  - Not all applicable network policies are using selectors
  - Not all applicable network policies are using ports
  - No monitors defined

⚠️  Warning Descriptions:
  - There is no common zarf.yaml file
  - Version is not consistent across flavors and package
  - Network policies with 'remoteGenerated: Anywhere' are present, review needed
  - No SSO configuration found, review needed
-----------------------------
Package: llama-cpp-python

⛔ Failures: 1

⛔ Failure Descriptions:
  - UDS Package CR does not exist for llama-cpp-python
-----------------------------
Package: text-embeddings

⛔ Failures: 1

⛔ Failure Descriptions:
  - UDS Package CR does not exist for text-embeddings
-----------------------------
Package: whisper

⛔ Failures: 1

⛔ Failure Descriptions:
  - UDS Package CR does not exist for whisper
-----------------------------
Package: supabase

❌ Errors: 5
⚠️  Warnings: 7

❌ Error Descriptions:
  - Manifests present in zarf.yaml
  - uds-config chart is invalid
  - Not all applicable network policies are using selectors
  - Not all applicable network policies are using ports
  - No monitors defined

⚠️  Warning Descriptions:
  - There is no common zarf.yaml file
  - Version is not consistent across flavors and package
  - Network policies with 'remoteGenerated: Anywhere' are present, review needed
  - SAML is not the default protocol, review needed
  - SSO client name not in preferred format, review needed
  - SSO client id not in the format uds-<group>-<application>-<protocol>, review needed
  - SSO secret name not in the format <application>-sso, review needed
-----------------------------
UDS Capability Issues

❌ Error Descriptions:
  - Not all pods have the istio sidecar
-----------------------------
@justinthelaw justinthelaw self-assigned this Sep 30, 2024
@justinthelaw justinthelaw changed the title chore: fixes on package manifests to obtain Made for UDS "Silver" chore: fix package manifests to obtain Made for UDS "Silver" Sep 30, 2024
@justinthelaw
Copy link
Contributor Author

Blocked by, depends on #1149

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant