Switch uni-delta job to use S3 API from Ceph

[maximsava12]

Glance/RBD is already covered by uni-gamma, so uni-delta could be changed to test glance/S3 instead, using the S3 API provided by ceph.

Depends-On: openstack-k8s-operators/ci-framework#3169

https://issues.redhat.com/browse/OSPRH-11411

[openshift-ci]

[APPROVALNOTIFIER] This PR is NOT APPROVED

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

[openshift-ci]

Hi @maximsava12. Thanks for your PR.

I'm waiting for a openstack-k8s-operators member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

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-sigs/prow repository.

[softwarefactory-project-zuul]

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/79fc87b45eff4ef597e8118de04286f2

✔️ noop SUCCESS in 0s
❌ rhoso-architecture-validate-uni04delta FAILURE in 5m 00s
❌ rhoso-architecture-validate-uni04delta-ipv6 FAILURE in 4m 27s

[softwarefactory-project-zuul]

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/1c2bebf73a784f338791c0a5c6951196

✔️ noop SUCCESS in 0s
✔️ rhoso-architecture-validate-uni04delta SUCCESS in 4m 02s
❌ rhoso-architecture-validate-uni04delta-ipv6 FAILURE in 4m 45s

[softwarefactory-project-zuul]

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/a926c527be8240ee81229d7218e2a006

✔️ noop SUCCESS in 0s
✔️ rhoso-architecture-validate-uni04delta SUCCESS in 3m 48s
❌ rhoso-architecture-validate-uni04delta-ipv6 FAILURE in 4m 29s

[softwarefactory-project-zuul]

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/f50e073907014bceb14a2dd11ac8bcbe

✔️ noop SUCCESS in 0s
✔️ rhoso-architecture-validate-uni04delta SUCCESS in 5m 33s
❌ rhoso-architecture-validate-uni04delta-ipv6 FAILURE in 5m 07s

[softwarefactory-project-zuul]

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/161cdd1d6033493eb60a127986acf0f2

✔️ noop SUCCESS in 0s
✔️ rhoso-architecture-validate-uni04delta SUCCESS in 5m 02s
❌ rhoso-architecture-validate-uni04delta-ipv6 FAILURE in 4m 32s

[softwarefactory-project-zuul]

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/6138b28c2f944d579ddf9672f30aaf9a

✔️ noop SUCCESS in 0s
❌ rhoso-architecture-validate-uni04delta FAILURE in 4m 32s
❌ rhoso-architecture-validate-uni04delta-ipv6 FAILURE in 4m 32s

[softwarefactory-project-zuul]

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/f45bdf7e00974987ae2482464ae73c27

✔️ noop SUCCESS in 0s
❌ rhoso-architecture-validate-uni04delta FAILURE in 3m 40s
❌ rhoso-architecture-validate-uni04delta-ipv6 FAILURE in 3m 28s

[softwarefactory-project-zuul]

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/fe0c63e7d2f2444d932f89057caae1f4

✔️ noop SUCCESS in 0s
✔️ rhoso-architecture-validate-uni04delta SUCCESS in 5m 44s
❌ rhoso-architecture-validate-uni04delta-ipv6 FAILURE in 5m 00s

[fmount]

Do we need this? I see you create it via ci-framework, so in theory we do not require this definition in architecture, but I might be wrong [1].

[1] openstack-k8s-operators/ci-framework#3169

[maximsava12]

Removed

[abays]

/ok-to-test

[abays]

Because of this new dependency, this PR depends on openstack-k8s-operators/ci-framework#3169

[fultonj]

Without this change:

[fultonj@stybba review]$ cd architecture/examples/dt/uni04delta/
[fultonj@stybba uni04delta{main}]$ 
[fultonj@stybba uni04delta{main}]$ kustomize build . | grep glance: -A 33
  glance:
    apiOverrides:
      default:
        route: {}
    template:
      customServiceConfig: |
        [DEFAULT]
        debug = True
        enabled_backends = default_backend:rbd

        [glance_store]
        default_backend = default_backend

        [default_backend]
        rbd_store_ceph_conf = /etc/ceph/ceph.conf
        store_description = "RBD backend"
        rbd_store_pool = images
        rbd_store_user = openstack
      databaseInstance: openstack
      glanceAPIs:
        default:
          apiTimeout: 600
          networkAttachments:
          - storage
          override:
            service:
              internal:
                metadata:
                  annotations:
                    metallb.universe.tf/address-pool: internalapi
                    metallb.universe.tf/allow-shared-ip: internalapi
                    metallb.universe.tf/loadBalancerIPs: 172.17.0.80
                spec:
                  type: LoadBalancer
[fultonj@stybba uni04delta{main}]$ 

With this change:

[fultonj@stybba uni04delta{unidelta-glance-s3-backend}]$ kustomize build . | grep glance: -A 33
  glance:
    apiOverrides:
      default:
        route:
          haproxy.router.openshift.io/timeout: 60s
    template:
      customServiceConfig: |
        [DEFAULT]
        debug=True
        enabled_backends = default_backend:s3
        [glance_store]
        default_backend = default_backend
        [default_backend]
        s3_store_create_bucket_on_put = True
        s3_store_bucket_url_format = "path"
      databaseInstance: openstack
      glanceAPIs:
        default:
          customServiceConfigSecrets:
          - s3glance
          networkAttachments:
          - storage
          override:
            service:
              internal:
                metadata:
                  annotations:
                    metallb.universe.tf/address-pool: internalapi
                    metallb.universe.tf/allow-shared-ip: internalapi
                    metallb.universe.tf/loadBalancerIPs: 172.17.0.80
                spec:
                  type: LoadBalancer
          replicas: 3
      preserveJobs: false
[fultonj@stybba uni04delta{unidelta-glance-s3-backend}]$ 

[fultonj]

I don't have any objections to this change. Depends-on needs to merge first.

[fultonj]

I'm fine with this change. Let's wait for ci-framework patch to merge before merging this one.

[fultonj]

@maximsava12 I've added the label do-not-merge/work-in-progress to this. After your ci-framework depends-on patch merges, please remove that label so we (arch team) can see it again in our weekly review.

[maximsava12]

/unhold

[maximsava12]

/hold cancel