Remove DISABLED value from 'zowe.verifyCertificates' property (v2.x)#4802
Remove DISABLED value from 'zowe.verifyCertificates' property (v2.x)#4802nxhafa wants to merge 5 commits into
Conversation
Signed-off-by: Nafi Xhafa <nafi.xhafa@broadcom.com>
|
build 10462 SUCCEEDED. |
|
Test workflow 8702 is started. |
Signed-off-by: Nafi Xhafa <nafi.xhafa@broadcom.com>
|
build 10490 SUCCEEDED. |
|
Test workflow 8715 is started. |
1000TurquoisePogs
left a comment
There was a problem hiding this comment.
You cannot do a breaking change without a major version bump, so it is not possible to do within v2.
Signed-off-by: Nafi Xhafa <nafi.xhafa@broadcom.com>
|
build 10500 SUCCEEDED. |
|
Test workflow 8718 is started. |
|
@1000TurquoisePogs This is the removal of HTTP that we discussed and agreed on on the TSC, I will bring it up again this Thursday as it seems that there were some misunderstandings. |
|
Removal of HTTP and removal of verifyCertificates=DISABLED are two different things. |
|
Test workflow 8718 is started. |
|
Test workflow 8719 is started. |
Signed-off-by: Nafi Xhafa <nafi.xhafa@broadcom.com>
|
build 10528 FAILED. |
2 similar comments
|
build 10528 FAILED. |
|
build 10528 FAILED. |
|
build 10578 FAILED. |
Please check if your PR fulfills the following requirements. This is simply a reminder of what we are going to look for before merging your PR. If you don't know all of this information when you create this PR, don't worry. You can edit this template as you're working on it.
PR type
What type of changes does your PR introduce to Zowe? Put an
xin the box that applies to this PR. If you're unsure about any of them, don't hesitate to ask.DISABLEDoption fromzowe.verifyCertificatesproperty in zowe.yaml. IfDISABLEDis provided, an error will be logged and zowe will not start.Relevant issues
Fixes
Changes proposed in this PR
zowe.verifyCertificatesare now onlySTRICTandNONSTRICTDoes this PR introduce a breaking change?
Does this PR add or change a YAML parameter?
Does this PR do something the person installing Zowe should know about?
DISABLEDoption fromzowe.verifyCertificatespropertymulti-line description
Is there a related doc issue or Pull Request?
Doc issue/PR number: zowe/docs-site #5051
Other information
zowe/api-layer PR number: #4683