Add common TypeInstance injection

[mszostok]

Description

Blocked by: #630

To make it easy to define injection of backend storage for TypeInstances with dynamic values, in Global and Action Policies we need to introduce an additional feature—common TypeInstance injection configuration:

interface:
  rules: [] # rules for Interfaces, now nested under `interface`

  default: # properties applied to all rules above
    inject:
      requiredTypeInstances:
        - id: "3ef2e4ac-9070-4093-a3ce-142139fd4a16"
          description: "Helm storage (cap.type.helm.storage:0.1.0)"

          # it works globally, not only for storage backends:
        - id: "c4f66896-c2a1-4031-b847-b55ae6a26c80"
          description: "GCP SA (cap.type.gcp.auth.service-account:0.1.0)"

typeInstance:
  rules: [] # ...

Acceptance Criteria

• Extends the Policy GraphQL syntax
• Update Policy GraphQL Go client (ensure that new policy can be saved and retrieved with update capact CLI)
• Update PolicyEnforceClient
  • Doesn't affect Implementation selection (do not change ListImplementationRevisionsForInterface method)
  • Support injecting arbitrary required TypeInstances
  • Support Storage backends:

    capact/pkg/hub/client/policy_enforced_client.go

    Lines 116 to 118 in 86f0be8

    	// TODO(https://github.com/capactio/capact/issues/624):
    	// 2. Global defaults based on required TypeInstance injection
    	// e.mergedPolicy.Interface.Defaults

  • Add unit-tests for all possible merging scenarios.
• Add integration test

  capact/test/e2e/action_test.go

  Line 145 in 86f0be8

  It("should pick Implementation B", func() {

  • First scenario, ensure that default injection works,
  • Second scenario, ensure that it can be overridden with Interface specific injection rule.
• Add documentation on website
  • Mention that defaults doesn't affect Implementation resolution

[mkuziemko]

Here is a list of PR to review:

• Add common TypeInstance injection #635
• Add policy description about common TypeInstance injection website#111

During testing, I also spotted a potential issue with one of the manifests from hub-manifest, I added a note in the PR and I will create a PR if it will be verified that is needed.