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.

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

Control in-kcp-mode flag using target flag #1663

Merged
merged 10 commits into from
May 17, 2023

Conversation

nesmabadr
Copy link
Contributor

@nesmabadr nesmabadr commented May 12, 2023

Description

Changes proposed in this pull request:

  • Patch --in-kcp-mode during LM deployment if target passed to Kyma alpha deploy is set to remote
  • Added unit test

Related issue(s)
Resolves kyma-project/lifecycle-manager#575

@nesmabadr nesmabadr requested a review from a team as a code owner May 12, 2023 10:42
@kyma-bot kyma-bot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label May 12, 2023
@nesmabadr nesmabadr marked this pull request as draft May 12, 2023 10:47
@kyma-bot kyma-bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label May 12, 2023
@nesmabadr nesmabadr marked this pull request as ready for review May 12, 2023 11:05
@kyma-bot kyma-bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label May 12, 2023
@nesmabadr nesmabadr requested a review from lindnerby May 16, 2023 10:45
@nesmabadr nesmabadr mentioned this pull request May 16, 2023
@nesmabadr nesmabadr requested a review from lindnerby May 17, 2023 07:16
@nesmabadr nesmabadr requested a review from lindnerby May 17, 2023 07:43
@kyma-bot kyma-bot added the lgtm Looks good to me! label May 17, 2023
@kyma-bot kyma-bot merged commit 07c8f08 into kyma-project:main May 17, 2023
kyma-bot pushed a commit that referenced this pull request May 24, 2023
* remove module template managed-by (#1657)

* Control in-kcp-mode flag using target flag (#1663)

* Control in-kcp-mode flag using target flag + add unit test

* remove mapstructure.Decode and use Deployment structure

* fix naming

* Check if target is remote

* change condition + PatchDeploymentWithInKcpModeFlag

* refactor applyManifests

* fix linting

* PR comments

* fix condition

* PR comments

* Make v1beta2 default (#1666)

* Replace references from v1beta1 to v1beta2

* Extend Kyma bootrap with setting sync label depending on kcp-mode

* Use kcp mode value for bootstrap

* bump hydroform image (#1660)

* Arm compiler (#1665)

* add Makefile command to compile on Apple Silicon

* add darwin-arm build instruction to global build

* reorder targets and add comments

* updated-dependencies:
- dependency-name: github.com/docker/distribution
  dependency-type: indirect

* Release 2.14.1

---------

Co-authored-by: Benjamin Lindner <50365642+lindnerby@users.noreply.github.com>
Co-authored-by: Nesma Badr <nesmabadr@aucegypt.edu>
Co-authored-by: Marcin Dobrochowski <39153236+anoipm@users.noreply.github.com>
Co-authored-by: Amritanshu Sikdar <amritanshu.sikdar@sap.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
kyma-bot pushed a commit that referenced this pull request May 30, 2023
* remove module template managed-by (#1657)

* Control in-kcp-mode flag using target flag (#1663)

* Control in-kcp-mode flag using target flag + add unit test

* remove mapstructure.Decode and use Deployment structure

* fix naming

* Check if target is remote

* change condition + PatchDeploymentWithInKcpModeFlag

* refactor applyManifests

* fix linting

* PR comments

* fix condition

* PR comments

* Make v1beta2 default (#1666)

* Replace references from v1beta1 to v1beta2

* Extend Kyma bootrap with setting sync label depending on kcp-mode

* Use kcp mode value for bootstrap

* bump hydroform image (#1660)

* Arm compiler (#1665)

* add Makefile command to compile on Apple Silicon

* add darwin-arm build instruction to global build

* reorder targets and add comments

* updated-dependencies:
- dependency-name: github.com/docker/distribution
  dependency-type: indirect

* Release 2.14.3

---------

Co-authored-by: Nesma Badr <nesmabadr@aucegypt.edu>
Co-authored-by: Marcin Dobrochowski <39153236+anoipm@users.noreply.github.com>
Co-authored-by: Amritanshu Sikdar <amritanshu.sikdar@sap.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm Looks good to me! size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Refactor target attribute to control --in-kcp-mode flag during deployment in CLI
3 participants