score-spec/score-helm

Score implementation for Helm (deprecated)

score-helm's Issues

[bug] Only 1 service is generated

Describe the bug
When running score-helm run -f score.yaml -o manifest.yaml, it generates a file that is used by the 'score-helm-charts/workload' helm chart. However, despite my score.yaml file containing 2 services, the helm-chart creates a service in k8s as 1 service with 2 pods.

To Reproduce

apiVersion: score.dev/v1b1

metadata:
  name: both

containers:
  backend:
    image: be:1.0.0

  frontend:
    image: fe:1.0.0

service:
  ports:
    frontend:
      port: 8080
      targetPort: 8080
    backend:
      port: 8000
      targetPort: 8000

resources:
  env:
    type: environment

  my-be-dns:
    type: dns

  my-fe-dns:
    type: dns

  bobby-bucket:
    type: s3

  backend-route:
    type: route
    params:
      path: /api
      port: 8000
      host: ${resources.my-be-dns.host}

  frontend-route:
    type: route
    params:
      path: /
      port: 8080
      host: ${resources.my-fe-dns.host}

score-helm run -f score.yaml -o manifest.yaml

helm install test-app score-helm-charts/workload --values ./manifest.yaml

Expected behavior
Would expect an 2 deployments within k8s - and 2 services running.

[bug] - score-helm: command not found

Seems like score-helm go install does not install binaries into GOBIN, please see this report - https://ci.sparrowhub.io/report/1836

[bug] execute go tests in CI

Describe the bug
Currently the github action is not executing the go tests for this project. This should be fixed. Failing tests should block the release and merge requests to the main branch.

To Reproduce
Steps to reproduce the behavior:

1. check the github action setup under release.yaml
2. it only executes Checkout, Fetch all tags, Set up Go and Run GoReleaser

Expected behavior
Failing tests should block the release and merge requests to the main branch.

[docs] Typo in README.md

A word, productivity, was misspelled as producticity.

• Go to Readme.md.
• Scroll to the Score overview section.
• See error in the first sentence. The sentence:

Score aims to improve developer producticity and experience by reducing the risk of configurtaion inconsistencies between local and remote environments.

Add image flag for . image convention

See score-spec/score-humanitec#69 for further context. To ensure consistency we'd like to implement this feature for score-helm as well.