Closed gcs278 closed 3 months ago
@gcs278: This pull request references Jira Issue OCPBUGS-22739, which is invalid:
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.
@gcs278: This pull request references Jira Issue OCPBUGS-32634, which is invalid:
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.
/jira refresh
@gcs278: This pull request references Jira Issue OCPBUGS-32634, which is invalid:
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.
/jira refresh
@gcs278: This pull request references Jira Issue OCPBUGS-32634, which is valid. The bug has been moved to the POST state.
Requesting review from QA contact: /cc @ShudiLi
/lgtm /approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: alebedev87
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/label qe-approved thanks
@gcs278: This pull request references Jira Issue OCPBUGS-32634, which is valid.
Requesting review from QA contact: /cc @ShudiLi
The backport introduces a change that prevents the router from crashing due to invalid rewrite-target annotations, so any risk is weighed against this benefit. Any remaining risk is eliminated because there is no change to valid rewrite-target annotations, only invalid rewrite-target annotations.
/label backport-risk-assessed
/lgtm
tested it with 4.14.0-0.ci.test-2024-05-17-033138-ci-ln-m43pm4t-latest
1.
% oc get clusterversion
NAME VERSION AVAILABLE PROGRESSING SINCE STATUS
version 4.14.0-0.ci.test-2024-05-17-033138-ci-ln-m43pm4t-latest True False 24m Cluster version is 4.14.0-0.ci.test-2024-05-17-033138-ci-ln-m43pm4t-latest
2.
% oc -n openshift-ingress rsh router-default-58d8855dfc-bj8nx
sh-4.4$ grep "http-request replace-path" haproxy-config.template
http-request replace-path ^{{ $cfg.Path }}/?(.*)$ '{{ processRewriteTarget $pathRewriteTarget }}'
http-request replace-path ^{{ $cfg.Path }}(.*)$ '{{ processRewriteTarget $pathRewriteTarget }}'
sh-4.4$
3.
% oc annotate route/unsec-server3 haproxy.router.openshift.io/rewrite-target="/t t$%&*(){}[].,z#:\`_-|" --overwrite
route.route.openshift.io/unsec-server3 annotate
4.
% oc get route unsec-server3 -o=jsonpath="{.metadata.annotations.haproxy\.router\.openshift\.io/rewrite-target}"
/t t$%&*(){}[].,z#:`_-|%
seems unrelated: /test e2e-upgrade
/retest-required
@ShudiLi Just checking on cherry-pick-approved
label since it's been a couple weeks.
/label cherry-pick-approved
@gcs278: 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/e2e-metal-ipi-ovn-ipv6 | a71e1a3a5f790be5f67d236478ac30508f416a25 | link | false | /test e2e-metal-ipi-ovn-ipv6 |
Full PR test history. Your PR dashboard.
Not related { OVNKubernetesResourceRetryFailure was at or above info for at least 56m48s on platformidentification.JobType{Release:"4.14", FromRelease:"", Platform:"aws", Architecture:"amd64", Network:"ovn", Topology:"ha"} (maxAllowed=0s): pending for 0s, firing for 56m48s:
/test e2e-aws-serial
@gcs278: Jira Issue OCPBUGS-32634: All pull requests linked via external trackers have merged:
Jira Issue OCPBUGS-32634 has been moved to the MODIFIED state.
[ART PR BUILD NOTIFIER]
This PR has been included in build ose-haproxy-router-base-container-v4.14.0-202405282308.p0.gc3a2430.assembly.stream.el8 for distgit ose-haproxy-router-base. All builds following this will include this PR.
Fix included in accepted release 4.14.0-0.nightly-2024-05-29-034539
/cherry-pick release-4.13
@gcs278: new pull request created: #608
This is an manual cherry-pick of https://github.com/openshift/router/pull/534
Process the
haproxy.router.openshift.io/rewrite-target
annotation value to prevent HaProxy configuration issues while maintaining API compatibility. Properly handle values with spaces, backslashes, and other special characters.Add a new package, rewritetarget, to organize the functions that process the rewrite target annotation. Add a separate unit test package, rewritetarget_test, to enforce black-box testing principles.