Add barbican commands to makefile

[jparoly]

Change-Id: Ie032ab5201c5fe08d86c9cb2077d97a5d13449b9

Depends-on: openstack-k8s-operators/barbican-operator#5

[cescgina]

Judging from the controller files from barbican-operator, the only dependency that will be need is infra, but will need later confirmation as it seems the operator it's in a very early stage (there are no images pushed to quay yet).

[openshift-ci]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: jparoly
Once this PR has been reviewed and has the lgtm label, please assign sandeepyadav93 for approval. For more information see the Kubernetes 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:

• OWNERS

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

[abays]

recheck

[jparoly]

/retest

[abays]

Until Barbican is added to the OpenStack operator bundle, this can fail in a fresh environment because the make deploy_cleanup will not have the Barbican CRDs available in the OCP cluster (CRDs that are installed via make openstack_crds, which reads from the OpenStack operator bundle). Just a heads-up.

[abays]

When I try make barbican_kuttl with this patch, I get the following errors during the KUTTL tests:

    logger.go:42: 14:55:12 | basic-deployment/1-deploy-barbican | The Barbican "barbican-sample" is invalid: 
    logger.go:42: 14:55:12 | basic-deployment/1-deploy-barbican | * spec.barbicanAPI.externalEndpoints[0].endpoint: Required value
    logger.go:42: 14:55:12 | basic-deployment/1-deploy-barbican | * spec.serviceAccount: Required value

[cescgina]

When I try make barbican_kuttl with this patch, I get the following errors during the KUTTL tests:

    logger.go:42: 14:55:12 | basic-deployment/1-deploy-barbican | The Barbican "barbican-sample" is invalid: 
    logger.go:42: 14:55:12 | basic-deployment/1-deploy-barbican | * spec.barbicanAPI.externalEndpoints[0].endpoint: Required value
    logger.go:42: 14:55:12 | basic-deployment/1-deploy-barbican | * spec.serviceAccount: Required value

I got the same error while trying make barbican_deploy looks like the sample from https://github.com/openstack-k8s-operators/barbican-operator/blob/main/config/samples/barbican_v1beta1_barbican.yaml is missing two required fields.

endpoint in https://github.com/openstack-k8s-operators/barbican-operator/blob/2ca654c53f1784d070761b01e9cee72473dfc362/config/samples/barbican_v1beta1_barbican.yaml#L48 should be set to some value and the spec should have a serviceAccount

[jparoly]

When I try make barbican_kuttl with this patch, I get the following errors during the KUTTL tests:

    logger.go:42: 14:55:12 | basic-deployment/1-deploy-barbican | The Barbican "barbican-sample" is invalid: 
    logger.go:42: 14:55:12 | basic-deployment/1-deploy-barbican | * spec.barbicanAPI.externalEndpoints[0].endpoint: Required value
    logger.go:42: 14:55:12 | basic-deployment/1-deploy-barbican | * spec.serviceAccount: Required value

I got the same error while trying make barbican_deploy looks like the sample from https://github.com/openstack-k8s-operators/barbican-operator/blob/main/config/samples/barbican_v1beta1_barbican.yaml is missing two required fields.

endpoint in https://github.com/openstack-k8s-operators/barbican-operator/blob/2ca654c53f1784d070761b01e9cee72473dfc362/config/samples/barbican_v1beta1_barbican.yaml#L48 should be set to some value and the spec should have a serviceAccount

operator is still being developed it seems, so this is blocked by: https://issues.redhat.com/browse/OSP-22827

[openshift-ci]

@jparoly: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name	Commit	Details	Required	Rerun command
ci/prow/install-yamls-deploy	2a3dd2f	link	false	/test install-yamls-deploy

Full PR test history. Your PR dashboard.

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/test-infra repository. I understand the commands that are listed here.

[openshift-merge-robot]

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/test-infra repository.

[dmendiza]

This PR can be closed now that #577 has merged.