Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Upgrade for k8s.io/client-go/transport #8705

Merged
merged 21 commits into from
Jun 9, 2020
Merged

Upgrade for k8s.io/client-go/transport #8705

merged 21 commits into from
Jun 9, 2020

Conversation

rafalpotempa
Copy link
Contributor

@rafalpotempa rafalpotempa commented Jun 4, 2020 •
edited
Loading

Description

Changes proposed in this pull request:

  • Upgraded k8s dependencies to 1.16.10 where it proved to be vulnerable
  • Application Operator migrated to go modules

Related issue(s)

app-operator migration related to #8417

@rafalpotempa rafalpotempa self-assigned this Jun 4, 2020
@netlify
Copy link

netlify bot commented Jun 4, 2020 •
edited
Loading

🥰 Documentation preview ready! 🥰

Built with commit 6baa697

https://deploy-preview-8705--kyma-project-docs-preview.netlify.app

@rafalpotempa rafalpotempa force-pushed the upgrade-transport branch 2 times, most recently from 14ee9bf to 3cf80d6 Compare June 4, 2020 11:37
@rafalpotempa
Copy link
Contributor Author

/test pre-kyma-components-application-operator

@rafalpotempa
Copy link
Contributor Author

/test pre-kyma-components-application-operator

@rafalpotempa rafalpotempa added the area/security Issues or PRs related to security label Jun 8, 2020
@rafalpotempa
Copy link
Contributor Author

/test pre-kyma-tests-connector-service-tests
/test pre-kyma-tests-end-to-end-backup
/test pre-kyma-tests-end-to-end-external-solution-integration
/test pre-kyma-tests-end-to-end-upgrade
/test pre-kyma-tests-event-bus
/test pre-kyma-tests-integration-api-gateway
/test pre-master-kyma-common
/test pre-master-kyma-gardener-azure-integration
/test pre-master-kyma-gke-central-connector
/test pre-master-kyma-gke-compass-integration
/test pre-master-kyma-gke-integration
/test pre-master-kyma-gke-upgrade
/test pre-master-kyma-gke-upgrade-e2e
/test pre-master-kyma-integration

@rafalpotempa
Copy link
Contributor Author

/test pre-master-kyma-gke-upgrade-e2e

@rafalpotempa
Copy link
Contributor Author

/test pre-master-kyma-gke-integration

@rafalpotempa
Copy link
Contributor Author

/test pre-master-kyma-gke-central-connector

@rafalpotempa
Copy link
Contributor Author

/test pre-master-kyma-gardener-azure-integration

@rafalpotempa
Copy link
Contributor Author

/test pre-master-kyma-gke-upgrade-e2e

@kyma-bot
Copy link
Contributor

kyma-bot commented Jun 8, 2020

@rafalpotempa: The following test failed, say /retest to rerun all failed tests:

Test name Commit Details Rerun command
pre-master-kyma-gke-upgrade-e2e 6baa697 link /test pre-master-kyma-gke-upgrade-e2e

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.

koala7659
koala7659 approved these changes Jun 9, 2020
View reviewed changes
Copy link
Contributor

@koala7659 koala7659 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@rafalpotempa rafalpotempa merged commit 772ad7b into kyma-project:master Jun 9, 2020
@rafalpotempa rafalpotempa deleted the upgrade-transport branch January 15, 2021 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@koala7659 koala7659 koala7659 approved these changes

@franpog859 franpog859 franpog859 approved these changes

@akgalwas akgalwas Awaiting requested review from akgalwas

@crabtree crabtree Awaiting requested review from crabtree

@janmedrek janmedrek Awaiting requested review from janmedrek

@Maladie Maladie Awaiting requested review from Maladie

@Szymongib Szymongib Awaiting requested review from Szymongib

@dbadura dbadura Awaiting requested review from dbadura

@kfurgol kfurgol Awaiting requested review from kfurgol

@kjaksik kjaksik Awaiting requested review from kjaksik

@PK85 PK85 Awaiting requested review from PK85

@pkosiec pkosiec Awaiting requested review from pkosiec

@tgorgol tgorgol Awaiting requested review from tgorgol

Assignees

@rafalpotempa rafalpotempa

Labels
area/security Issues or PRs related to security
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@rafalpotempa @kyma-bot @koala7659 @franpog859