search

openstack-k8s-operators / openstack-operator

Meta Operator for OpenStack
https://openstack-k8s-operators.github.io/openstack-operator/
Apache License 2.0
27 stars 76 forks source link

gofmt fixes #900

Closed slagle closed 2 months ago

slagle commented 3 months ago

fixes from running "make gofmt"

Signed-off-by: James Slagle jslagle@redhat.com

openshift-ci[bot] commented 3 months ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: slagle

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

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/openstack-k8s-operators/openstack-operator/blob/main/OWNERS)~~ [slagle] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
bshephar commented 3 months ago

Kuttl failed with:

logger.go:42: 22:19:50 |  | Error from server (InternalError): error when creating "https://raw.githubusercontent.com/openstack-k8s-operators/infra-operator/main/config/samples/network_v1beta1_netconfig.yaml": Internal error occurred: failed calling webhook "mnetconfig.kb.io": failed to call webhook: Post "[https://infra-operator-controller-manager-service.openstack-operators.svc:443/mutate-network-openstack-org-v1beta1-netconfig?timeout=10s](https://infra-operator-controller-manager-service.openstack-operators.svc/mutate-network-openstack-org-v1beta1-netconfig?timeout=10s)": no endpoints available for service "infra-operator-controller-manager-service"

Let' recheck it

bshephar commented 3 months ago

/test openstack-operator-build-deploy-kuttl

stuggi commented 2 months ago

in the meantime this was fixed as part of https://github.com/openstack-k8s-operators/openstack-operator/commit/aa9196a5b8884ef0f8a3368fe933f6faf373d2e4

slagle commented 2 months ago

in the meantime this was fixed as part of aa9196a

ACK.