[release-4.10] OCPBUGS-5112: OpenStack: Force JSON content-type in Swift object listing by openshift-cherrypick-robot · Pull Request #6726 · openshift/installer

openshift-cherrypick-robot · 2022-12-21T13:58:43Z

This is an automated cherry-pick of #6725

/assign pierreprinetti

Some OpenStack object storages respond with `204 No Content` to list requests when there are no containers or objects to list. In these cases, when responding to requests with an `Accept: text/plain` or no `Accept` header, some object storages omit the `content-type` header in their status-204 responses. Now, Gophercloud throws an error when the response does not contain a `content-type` header. With this change, we work around the issue by forcing Gophercloud to request a JSON response from the object storage when listing objects. When passed an `Accept: application/json` header, the server responds with `200 Ok` and a `content-type` header in our tests. This solution gives us a fix that is easily backportable because it doesn't require any dependency bump.

openshift-ci-robot · 2022-12-21T13:58:50Z

@openshift-cherrypick-robot: Jira Issue OCPBUGS-5078 has been cloned as Jira Issue OCPBUGS-5112. Retitling PR to link against new bug.
/retitle [release-4.10] OCPBUGS-5112: OpenStack: Force JSON content-type in Swift object listing

Details

In response to this:

This is an automated cherry-pick of #6725

/assign pierreprinetti

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.

openshift-ci-robot · 2022-12-21T13:59:21Z

@openshift-cherrypick-robot: This pull request references Jira Issue OCPBUGS-5112, which is invalid:

expected dependent Jira Issue OCPBUGS-5078 to be in one of the following states: VERIFIED, RELEASE PENDING, CLOSED (ERRATA), CLOSED (CURRENT RELEASE), CLOSED (DONE), but it is MODIFIED instead

Comment /jira refresh to re-evaluate validity if changes to the Jira bug are made, or edit the title of this pull request to link to a different bug.

The bug has been updated to refer to the pull request using the external bug tracker.

Details

In response to this:

This is an automated cherry-pick of #6725

/assign pierreprinetti

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.

openshift-ci · 2022-12-21T13:59:58Z

@openshift-cherrypick-robot: No Bugzilla bug is referenced in the title of this pull request.
To reference a bug, add 'Bug XXX:' to the title of this pull request and request another bug refresh with /bugzilla refresh.

Details

In response to this:

[release-4.10] OCPBUGS-5112: OpenStack: Force JSON content-type in Swift object listing

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.

openshift-ci · 2022-12-21T17:10:29Z

@openshift-cherrypick-robot: The following tests 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/e2e-azurestack-upi	`1e99fb7`	link	false	`/test e2e-azurestack-upi`
ci/prow/okd-e2e-vsphere	`1e99fb7`	link	false	`/test okd-e2e-vsphere`
ci/prow/e2e-azurestack	`1e99fb7`	link	false	`/test e2e-azurestack`
ci/prow/e2e-alibaba	`1e99fb7`	link	false	`/test e2e-alibaba`

Full PR test history. Your PR dashboard.

Details

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.

pierreprinetti · 2022-12-21T21:44:34Z

/jira refresh
/approve
/lgtm

openshift-ci-robot · 2022-12-21T21:44:40Z

@pierreprinetti: This pull request references Jira Issue OCPBUGS-5112, which is valid. The bug has been moved to the POST state.

6 validation(s) were run on this bug

bug is open, matching expected state (open)
bug target version (4.10.z) matches configured target version for branch (4.10.z)
bug is in the state New, which is one of the valid states (NEW, ASSIGNED, POST)
dependent bug Jira Issue OCPBUGS-5078 is in the state Verified, which is one of the valid states (VERIFIED, RELEASE PENDING, CLOSED (ERRATA), CLOSED (CURRENT RELEASE), CLOSED (DONE))
dependent Jira Issue OCPBUGS-5078 targets the "4.11.z" version, which is one of the valid target versions: 4.11.0, 4.11.z
bug has dependents

Requesting review from QA contact:
/cc @eurijon

Details

In response to this:

/jira refresh
/approve
/lgtm

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.

mandre

/lgtm
/approve
/label backport-risk-assessed

eurijon · 2022-12-22T12:31:04Z

/label cherry-pick-approved

openshift-ci · 2022-12-22T12:31:35Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mandre, pierreprinetti

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

The pull request process is described here

Details

Needs approval from an approver in each of these files:

~~pkg/destroy/openstack/OWNERS~~ [mandre,pierreprinetti]

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

openshift-ci-robot · 2022-12-22T12:35:26Z

@openshift-cherrypick-robot: All pull requests linked via external trackers have merged:

openshift/installer#6726

Jira Issue OCPBUGS-5112 has been moved to the MODIFIED state.

Details

In response to this:

This is an automated cherry-pick of #6725

/assign pierreprinetti

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.

openshift-cherrypick-robot mentioned this pull request Dec 21, 2022

[release-4.11] OCPBUGS-5078: OpenStack: Force JSON content-type in Swift object listing #6725

Merged

openshift-cherrypick-robot assigned pierreprinetti Dec 21, 2022

openshift-ci Bot changed the title ~~[release-4.10] OCPBUGS-5078: OpenStack: Force JSON content-type in Swift object listing~~ [release-4.10] OCPBUGS-5112: OpenStack: Force JSON content-type in Swift object listing Dec 21, 2022

openshift-ci-robot added the jira/severity-low Referenced Jira bug's severity is low for the branch this PR is targeting. label Dec 21, 2022

openshift-ci-robot added the jira/invalid-bug Indicates that a referenced Jira bug is invalid for the branch this PR is targeting. label Dec 21, 2022

openshift-ci Bot requested review from EmilienM and stephenfin December 21, 2022 14:02

openshift-ci-robot added jira/valid-bug Indicates that a referenced Jira bug is valid for the branch this PR is targeting. bugzilla/valid-bug Indicates that a referenced Bugzilla bug is valid for the branch this PR is targeting. labels Dec 21, 2022

openshift-ci-robot removed the jira/invalid-bug Indicates that a referenced Jira bug is invalid for the branch this PR is targeting. label Dec 21, 2022

openshift-ci Bot requested a review from eurijon December 21, 2022 21:44

openshift-ci Bot added lgtm Indicates that a PR is ready to be merged. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Dec 21, 2022

mandre reviewed Dec 22, 2022

View reviewed changes

openshift-ci Bot added the backport-risk-assessed Indicates a PR to a release branch has been evaluated and considered safe to accept. label Dec 22, 2022

openshift-ci Bot assigned mandre Dec 22, 2022

openshift-ci Bot added the cherry-pick-approved Indicates a cherry-pick PR into a release branch has been approved by the release branch manager. label Dec 22, 2022

openshift-merge-robot merged commit dd5f512 into openshift:release-4.10 Dec 22, 2022

pierreprinetti mentioned this pull request Jan 20, 2023

OCPBUGS-6086: OpenStack: Force JSON content-type in Swift #6787

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[release-4.10] OCPBUGS-5112: OpenStack: Force JSON content-type in Swift object listing#6726

[release-4.10] OCPBUGS-5112: OpenStack: Force JSON content-type in Swift object listing#6726
openshift-merge-robot merged 1 commit intoopenshift:release-4.10from
openshift-cherrypick-robot:cherry-pick-6725-to-release-4.10

openshift-cherrypick-robot commented Dec 21, 2022

Uh oh!

openshift-ci-robot commented Dec 21, 2022

Uh oh!

openshift-ci-robot commented Dec 21, 2022

Uh oh!

openshift-ci Bot commented Dec 21, 2022

Uh oh!

openshift-ci Bot commented Dec 21, 2022

Uh oh!

pierreprinetti commented Dec 21, 2022

Uh oh!

openshift-ci-robot commented Dec 21, 2022

Uh oh!

mandre left a comment

Uh oh!

eurijon commented Dec 22, 2022

Uh oh!

openshift-ci Bot commented Dec 22, 2022

Uh oh!

openshift-ci-robot commented Dec 22, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

Conversation

openshift-cherrypick-robot commented Dec 21, 2022

Uh oh!

openshift-ci-robot commented Dec 21, 2022

Uh oh!

openshift-ci-robot commented Dec 21, 2022

Uh oh!

openshift-ci Bot commented Dec 21, 2022

Uh oh!

openshift-ci Bot commented Dec 21, 2022

Uh oh!

pierreprinetti commented Dec 21, 2022

Uh oh!

openshift-ci-robot commented Dec 21, 2022

Uh oh!

mandre left a comment

Choose a reason for hiding this comment

Uh oh!

eurijon commented Dec 22, 2022

Uh oh!

openshift-ci Bot commented Dec 22, 2022

Uh oh!

openshift-ci-robot commented Dec 22, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants