Skip to content
This repository has been archived by the owner on Jul 4, 2024. It is now read-only.

Kyma 2.4.3 #3301

Merged
merged 9 commits into from
Sep 15, 2023
Merged

Kyma 2.4.3 #3301

merged 9 commits into from
Sep 15, 2023

Conversation

ognyvrac
Copy link
Contributor

@ognyvrac ognyvrac commented Sep 13, 2023 •
edited
Loading

Description
Update Kyma to 2.4.3

Changes proposed in this pull request:

  • Update Kyma CLI in benchmark and provision scripts
  • Move patch logic to Helm overrides
  • Change istio images to external ones as Istio has been updated to 1.14.1; With Istio 1.15 ARM images should be supported out of the box, that is why we have not downloaded the external ones and maintained them locally
  • Remove Kyma workplace changes that were needed to exclude outdated resources, kept reusability for local scripts

Related issue(s)

  • ...

Pull Request status

  • Implementation
  • Unit tests
  • Integration tests
  • chart/compass/values.yaml is updated
  • Mocks are regenerated, using the automated script

@ognyvrac ognyvrac requested review from a team as code owners September 13, 2023 12:11
@kyma-bot kyma-bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. 🦖 team-raptor Team Raptor Label labels Sep 13, 2023
@ognyvrac ognyvrac changed the title Kyma 2.4.3 [WIP]Kyma 2.4.3 Sep 13, 2023
@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 Sep 13, 2023
@ognyvrac
Copy link
Contributor Author

/test pre-main-compass-gke-benchmark

@ognyvrac
Copy link
Contributor Author

/hold

@kyma-bot kyma-bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Sep 14, 2023
@ognyvrac ognyvrac changed the title [WIP]Kyma 2.4.3 Kyma 2.4.3 Sep 14, 2023
@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 Sep 14, 2023
@ognyvrac ognyvrac added the 👋 request review Review required label Sep 14, 2023
@ognyvrac ognyvrac requested a review from a team as a code owner September 14, 2023 12:18
@kyma-bot kyma-bot added the do-not-merge/missing-docs-review Indicates that a PR should not merge beacuse it has missing documentation review. label Sep 14, 2023
@PetarTodorovv
Copy link
Collaborator

/retest

@kyma-bot kyma-bot added lgtm Looks good to me! approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Sep 15, 2023
Peteva
Peteva approved these changes Sep 15, 2023
View reviewed changes
Copy link

@Peteva Peteva left a comment

Choose a reason for hiding this comment

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

+1

@kyma-bot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: PetarTodorovv, Peteva

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

@PetarTodorovv PetarTodorovv removed do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. 👋 request review Review required labels Sep 15, 2023
@kyma-bot kyma-bot merged commit 2b858fd into main Sep 15, 2023
@PetarTodorovv PetarTodorovv deleted the kyma-2.4.3 branch September 15, 2023 13:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Peteva Peteva Peteva approved these changes

@PetarTodorovv PetarTodorovv PetarTodorovv approved these changes

@alextargov alextargov Awaiting requested review from alextargov

@ivantenevvasilev ivantenevvasilev Awaiting requested review from ivantenevvasilev

Assignees

@PetarTodorovv PetarTodorovv

@Peteva Peteva

Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. do-not-merge/missing-docs-review Indicates that a PR should not merge beacuse it has missing documentation review. lgtm Looks good to me! size/L Denotes a PR that changes 100-499 lines, ignoring generated files. 🦖 team-raptor Team Raptor Label
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@ognyvrac @PetarTodorovv @kyma-bot @Peteva