8000 feat: skip ssl verify for gitlab by kz33 · Pull Request #1166 · caicloud/cyclone · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

feat: skip ssl verify for gitlab #1166

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 28, 2019
Merged

Conversation

kz33
Copy link
@kz33 kz33 commented Jun 28, 2019

What this PR does / why we need it:

Add your description

Which issue(s) this PR is related to (optional, link to 3rd issue(s)):

Fixes #

Reference to # https://github.com/caicloud/service/issues/761

Special notes for your reviewer:

/cc @zhujian7

Release note:

NONE

@caicloud-bot caicloud-bot added the release-note-none Denotes a PR that doesn't merit a release note. label Jun 28, 2019
@caicloud-bot caicloud-bot requested a review from zhujian7 June 28, 2019 05:50
@caicloud-bot caicloud-bot added caicloud-cla: yes Indicates the PR's author has not signed the Caicloud CLA. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Jun 28, 2019
@zhujian7
Copy link
Collaborator

/lgtm

@caicloud-bot caicloud-bot added the lgtm Indicates that a PR is ready to be merged. label Jun 28, 2019
@caicloud-bot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: zhujian7

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:

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

@caicloud-bot caicloud-bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 28, 2019
@zhujian7
Copy link
Collaborator

@supereagle should I cherrypick it to the master branch?

@caicloud-bot caicloud-bot merged commit 9cfad9a into caicloud:release-1.0 Jun 28, 2019
@zhujian7
Copy link
Collaborator

/cherrypick master

@caicloud-bot
Copy link
Collaborator

@zhujian7: new pull request created: #1190

In response to this:

/cherrypick master

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.

@supereagle
Copy link
Member

@supereagle should I cherrypick it to the master branch?

SGTM

@zhujian7 zhujian7 changed the title fest: skip ssl verify for gitlab feat: skip ssl verify for gitlab Jul 19, 2019
moshuipan pushed a commit to moshuipan/cyclone that referenced this pull request Jul 30, 2019
caicloud-bot pushed a commit that referenced this pull request Jul 31, 2019
* fix: update tenant pvc not found error (#1150)

* fix: argument escape problem (#1152)

* fix: resources not freed immediately after stage finished (#1157) (#1160)

* fix: resources not freed immediately after stage finished

* fix: remove retry config

* fest: skip ssl verify for gitlab (#1166)

* fix: last line log would be missing if not ended with \n (#1171)

* [release-1.0] fix: assign resource requirements of cyclone containers with 0 (#1169)

* fix: assign resource requirements of cyclone containers with 0

* test: add some test for apply resource requirements

* fix: use mkdir to acquire lock in git resolver (#1170)

* perf: improve git resolver performance with --depth (#1180)

* fix: sonar qube entrypoint bug fixes (#1183)

* chore: add debug logs for sonar qube image (#1182)

* [release-1.0] All container logs (#1184)

* feat: support all container logs stream and download

* chore: add unit test for folder logs

* fix: output container name

* fix: add auth info when get scan results (#1185)

* chore: add supported kubernetes versions (#1186)

* fix: low level bug of gitlab v3 version (#1188)

* chore: disable PVC watcher (#1191)

* [release-1.0] fix: skip ssl verify for gitlab (#1196)

* fix(kaide): skip ssl verify for gitlab

* fix(kaide): skip ssl verify for gitlab again

* fix(kaide): skip ssl verify for gitlab again and again
tosone pushed a commit to tosone/cyclone that referenced this pull request Dec 13, 2019
* fix: update tenant pvc not found error (cai
10000
cloud#1150)

* fix: argument escape problem (caicloud#1152)

* fix: resources not freed immediately after stage finished (caicloud#1157) (caicloud#1160)

* fix: resources not freed immediately after stage finished

* fix: remove retry config

* fest: skip ssl verify for gitlab (caicloud#1166)

* fix: last line log would be missing if not ended with \n (caicloud#1171)

* [release-1.0] fix: assign resource requirements of cyclone containers with 0 (caicloud#1169)

* fix: assign resource requirements of cyclone containers with 0

* test: add some test for apply resource requirements

* fix: use mkdir to acquire lock in git resolver (caicloud#1170)

* perf: improve git resolver performance with --depth (caicloud#1180)

* fix: sonar qube entrypoint bug fixes (caicloud#1183)

* chore: add debug logs for sonar qube image (caicloud#1182)

* [release-1.0] All container logs (caicloud#1184)

* feat: support all container logs stream and download

* chore: add unit test for folder logs

* fix: output container name

* fix: add auth info when get scan results (caicloud#1185)

* chore: add supported kubernetes versions (caicloud#1186)

* fix: low level bug of gitlab v3 version (caicloud#1188)

* chore: disable PVC watcher (caicloud#1191)

* [release-1.0] fix: skip ssl verify for gitlab (caicloud#1196)

* fix(kaide): skip ssl verify for gitlab

* fix(kaide): skip ssl verify for gitlab again

* fix(kaide): skip ssl verify for gitlab again and again
caicloud-bot pushed a commit that referenced this pull request Dec 17, 2019
* fix: update tenant pvc not found error (#1150)

* fix: argument escape problem (#1152)

* fix: resources not freed immediately after stage finished (#1157) (#1160)

* fix: resources not freed immediately after stage finished

* fix: remove retry config

* fest: skip ssl verify for gitlab (#1166)

* fix: last line log would be missing if not ended with \n (#1171)

* [release-1.0] fix: assign resource requirements of cyclone containers with 0 (#1169)

* fix: assign resource requirements of cyclone containers with 0

* test: add some test for apply resource requirements

* fix: use mkdir to acquire lock in git resolver (#1170)

* perf: improve git resolver performance with --depth (#1180)

* fix: sonar qube entrypoint bug fixes (#1183)

* chore: add debug logs for sonar qube image (#1182)

* [release-1.0] All container logs (#1184)

* feat: support all container logs stream and download

* chore: add unit test for folder logs

* fix: output container name

* fix: add auth info when get scan results (#1185)

* chore: add supported kubernetes versions (#1186)

* fix: low level bug of gitlab v3 version (#1188)

* chore: disable PVC watcher (#1191)

* [release-1.0] fix: skip ssl verify for gitlab (#1196)

* fix(kaide): skip ssl verify for gitlab

* fix(kaide): skip ssl verify for gitlab again

* fix(kaide): skip ssl verify for gitlab again and again

* fix: github release event webhook trigger workflow twice (#1202)

* fix: make dind bip configurable (#1207)

* feat: make resource requirement of gc pod configurable (#1206)

* fix: can not list working pods (#1224)

* feat: collect realtime init container logs (#1226)

* [release-1.0] feat(toolbox): add fstream tool to collect init container logs (#1227)

* chore(vendor): add spf13 cobra

* feat(toolbox): add fstream tool to collect init container logs

* fix: workflow run status check logic (#1228)

* fix(bitbucket): need response to do error check (#1232)

* fix(error code): clarify error code (#1233)

* chore: update gradle image version to 5.5.1 for stage template (#1229)

* fix: label value for wft event repo too long (#1230)

* fix(git-resolver): can not pull bitbucket code with personal access token (#1231)

* fix: workflow run status check logic (#1234)

* fix: set http.postBuffer for git resolver (#1225)

* fix: GitLab tag deletion should not trigger workflow (#1238)

* fix: bitbucket pr webhook problem (#1239)

* fix: can not list gitlab branch whose project name contains dot (#1235)

* [release-1.0] feat(server): support dry-run for creating integration (#1241)

* feat(server): add api for integration validation

* feat(server): dry-run for creating integration

* fix: validate bitbucket token (#1237)

* fix: gitlab version detect problem (#1236)

* [release-1.0] fix(log-stream): websocket interrupted occasionally (#1242)

* fix(log-stream): websocket interrupted occasionally

* refactor: send log stream by websocket

* fix: use error with reason for token validation fail (#1243)

* feat(api): support list pull requests of scm repo (#1240)

* fix: use authorization error to token validation fail (#1244)

* fix: init pull request status when wfr start (#1246)

* [release-1.0] fix: send notification only once (#1248)

* fix: send notification only once

* fix: no need to use lock for controller

* fix: specify some dial tcp errors (#1255)

* fix(panic): call of reflect.Value.IsNil on zero Value (#1256)

* fix: git clone from bitbucket failed (#1257)

* fix: defer exit to collect init containers' logs (#1258)

* fix: add target branch for list pull requests api (#1259)

* fix: out of range panic protection (#1261)

* fix: image tag wrong when workflow triggered by bitbucket release (#1264)

* [release-1.0] feat: add execution contexts api (#1267)

* feat: add precheck api

* feat: add list execution contexts api

* fix: add Closed phase for execution context

* [release-1.0] fix: GitHub tag trigger problem (#1270)

* feat: support tag create event for github webhook

* fix: delete release trigger for github

* test: add tests for pvc watcher (#1271)

* chore: add mount propagation (#1275)

* [cps-2.8] fix: close websocket when workflow run running completed (#1283)

* fix: close websocket when workflow run running completed

* refactor: check wfr regularly to reduce cost

* fix: eof of stage logs FolderReader (#1285)

* fix(1292): fix out of memory of golangci-lint (#1295)

* fix: reduce cpu usage caused by go ticker incorrect use (#1322)

* [cps-2.8] fix: deleting pod stuck on Terminating status (#1323)

* fix: deleting pod stuck on Terminating status

WorkflowRun GC SHOULD Wait all workflowRun related workload pods
deleting completed, then start gc pod to clean date on PV.Otherwise,
if the path which is used by workload pods in the PV is deleted
before workload pods deletion, the pod deletion process will get
stuck on Terminating status.

* fix(gc): deleting pod stuck on Terminating status

If there are pods not finished and this is not the last gc try
process, cyclone will not start gc pod to clean data on PV

*  feat(oauth): cherry-pick from v0.8 to v0.9 (#1165)

* feat(oauth): cherry-pick from v0.8 to v0.9

* feat(oauth): cherry-pick from v0.8 to v0.9

* fix(oauth): fix golint

* fix(oauth): delete unused code

* fix(oauth): format import path

* feat(oauth): cherry-pick from v0.9.5 to v0.9.7-alpha (#1209)

* fix: update tenant pvc not found error (#1150)

* fix: argument escape problem (#1152)

* fix: resources not freed immediately after stage finished (#1157) (#1160)

* fix: resources not freed immediately after stage finished

* fix: remove retry config

* fest: skip ssl verify for gitlab (#1166)

* fix: last line log would be missing if not ended with \n (#1171)

* [release-1.0] fix: assign resource requirements of cyclone containers with 0 (#1169)

* fix: assign resource requirements of cyclone containers with 0

* test: add some test for apply resource requirements

* fix: use mkdir to acquire lock in git resolver (#1170)

* perf: improve git resolver performance with --depth (#1180)

* fix: sonar qube entrypoint bug fixes (#1183)

* chore: add debug logs for sonar qube image (#1182)

* [release-1.0] All container logs (#1184)

* feat: support all container logs stream and download

* chore: add unit test for folder logs

* fix: output container name

* fix: add auth info when get scan results (#1185)

* chore: add supported kubernetes versions (#1186)

* fix: low level bug of gitlab v3 version (#1188)

* chore: disable PVC watcher (#1191)

* [release-1.0] fix: skip ssl verify for gitlab (#1196)

* fix(kaide): skip ssl verify for gitlab

* fix(kaide): skip ssl verify for gitlab again

* fix(kaide): skip ssl verify for gitlab again and again
caicloud-bot added a commit that referenced this pull request Apr 16, 2020
…oem-hr-4 (#1397)

* fix: update tenant pvc not found error (#1150)

* fix: argument escape problem (#1152)

* fix: resources not freed immediately after stage finished (#1157) (#1160)

* fix: resources not freed immediately after stage finished

* fix: remove retry config

* fest: skip ssl verify for gitlab (#1166)

* fix: last line log would be missing if not ended with \n (#1171)

* [release-1.0] fix: assign resource requirements of cyclone containers with 0 (#1169)

* fix: assign resource requirements of cyclone containers with 0

* test: add some test for apply resource requirements

* fix: use mkdir to acquire lock in git resolver (#1170)

* perf: improve git resolver performance with --depth (#1180)

* fix: sonar qube entrypoint bug fixes (#1183)

* chore: add debug logs for sonar qube image (#1182)

* [release-1.0] All container logs (#1184)

* feat: support all container logs stream and download

* chore: add unit test for folder logs

* fix: output container name

* fix: add auth info when get scan results (#1185)

* chore: add supported kubernetes versions (#1186)

* fix: low level bug of gitlab v3 version (#1188)

* chore: disable PVC watcher (#1191)

* [release-1.0] fix: skip ssl verify for gitlab (#1196)

* fix(kaide): skip ssl verify for gitlab

* fix(kaide): skip ssl verify for gitlab again

* fix(kaide): skip ssl verify for gitlab again and again

* fix: github release event webhook trigger workflow twice (#1202)

* fix: make dind bip configurable (#1207)

* feat: make resource requirement of gc pod configurable (#1206)

* fix: can not list working pods (#1224)

* feat: collect realtime init container logs (#1226)

* [release-1.0] feat(toolbox): add fstream tool to collect init container logs (#1227)

* chore(vendor): add spf13 cobra

* feat(toolbox): add fstream tool to collect init container logs

* fix: workflow run status check logic (#1228)

* fix(bitbucket): need response to do error check (#1232)

* fix(error code): clarify error code (#1233)

* chore: update gradle image version to 5.5.1 for stage template (#1229)

* fix: label value for wft event repo too long (#1230)

* fix(git-resolver): can not pull bitbucket code with personal access token (#1231)

* fix: workflow run status check logic (#1234)

* fix: set http.postBuffer for git resolver (#1225)

* fix: GitLab tag deletion should not trigger workflow (#1238)

* fix: bitbucket pr webhook problem (#1239)

* fix: can not list gitlab branch whose project name contains dot (#1235)

* [release-1.0] feat(server): support dry-run for creating integration (#1241)

* feat(server): add api for integration validation

* feat(server): dry-run for creating integration

* fix: validate bitbucket token (#1237)

* fix: gitlab version detect problem (#1236)

* [release-1.0] fix(log-stream): websocket interrupted occasionally (#1242)

* fix(log-stream): websocket interrupted occasionally

* refactor: send log stream by websocket

* fix: use error with reason for token validation fail (#1243)

* feat(api): support list pull requests of scm repo (#1240)

* fix: use authorization error to token validation fail (#1244)

* fix: init pull request status when wfr start (#1246)

* [release-1.0] fix: send notification only once (#1248)

* fix: send notification only once

* fix: no need to use lock for controller

* fix: specify some dial tcp errors (#1255)

* fix(panic): call of reflect.Value.IsNil on zero Value (#1256)

* fix: git clone from bitbucket failed (#1257)

* fix: defer exit to collect init containers' logs (#1258)

* fix: add target branch for list pull requests api (#1259)

* fix: out of range panic protection (#1261)

* fix: image tag wrong when workflow triggered by bitbucket release (#1264)

* [release-1.0] feat: add execution contexts api (#1267)

* feat: add precheck api

* feat: add list execution contexts api

* fix: add Closed phase for execution context

* [release-1.0] fix: GitHub tag trigger problem (#1270)

* feat: support tag create event for github webhook

* fix: delete release trigger for github

* test: add tests for pvc watcher (#1271)

* chore: add mount propagation (#1275)

* [cps-2.8] fix: close websocket when workflow run running completed (#1283)

* fix: close websocket when workflow run running completed

* refactor: check wfr regularly to reduce cost

* fix: eof of stage logs FolderReader (#1285)

* fix(1292): fix out of memory of golangci-lint (#1295)

* fix: reduce cpu usage caused by go ticker incorrect use (#1322)

* [cps-2.8] fix: deleting pod stuck on Terminating status (#1323)

* fix: deleting pod stuck on Terminating status

WorkflowRun GC SHOULD Wait all workflowRun related workload pods
deleting completed, then start gc pod to clean date on PV.Otherwise,
if the path which is used by workload pods in the PV is deleted
before workload pods deletion, the pod deletion process will get
stuck on Terminating status.

* fix(gc): deleting pod stuck on Terminating status

If there are pods not finished and this is not the last gc try
process, cyclone will not start gc pod to clean data on PV

* fix: svn checkout with can't convert string from 'UTF-8' to native encoding (#1357)

* feat(server): make qps and brust configurable for k8s client (#1359)

Co-authored-by: caicloud-bot <register@caicloud.io>
Co-authored-by: Jian Zhu <36154065+zhujian7@users.noreply.github.com>
Co-authored-by: Jiekun Zhang <zhangjiekun@caicloud.io>
Co-authored-by: De Chen <chende@caicloud.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. caicloud-cla: yes Indicates the PR's author has not signed the Caicloud CLA. lgtm Indicates that a PR is ready to be merged. release-note-none Denotes a PR that doesn't merit a release note. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0