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

Moved api pkg from eventing-controller #260

Merged
merged 6 commits into from
Nov 27, 2023

Conversation

mfaizanse
Copy link
Member

Description

Changes proposed in this pull request are mentioned here.

Related issue(s)

@mfaizanse mfaizanse requested review from a team as code owners November 20, 2023 16:37
@kyma-bot kyma-bot added the area/documentation Issues or PRs related to documentation label Nov 20, 2023
@mfaizanse mfaizanse added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. and removed area/documentation Issues or PRs related to documentation labels Nov 20, 2023
@kyma-bot kyma-bot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Nov 20, 2023
utils.WithEventingStreamData("Memory", "650M", 5, 5000),
utils.WithEventingEventTypePrefix("sap.kyma.custom"),
)
givenEventing := &v1alpha1.Eventing{
Copy link
Member Author

Choose a reason for hiding this comment

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

This is done to fix import cycle!!!

@mfaizanse mfaizanse linked an issue Nov 20, 2023 that may be closed by this pull request
2 tasks
| **publisher.​resources.​claims** | \[\]object | Claims lists the names of resources, defined in spec.resourceClaims, that are used by this container. This is an alpha field and requires enabling the DynamicResourceAllocation feature gate. This field is immutable. It can only be set for containers. |
| **publisher.​resources.​claims.​name** (required) | string | Name must match the name of one entry in pod.spec.resourceClaims of the Pod where this field is used. It makes that resource available inside a container. |
| **publisher.​resources.​limits** | map\[string\]\{integer or string\} | Limits describes the maximum amount of compute resources allowed. More info: https://kubernetes.io/docs/concepts/configuration/manage-resources-containers/ |
| Parameter | Type | Description |
Copy link
Member Author

Choose a reason for hiding this comment

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

Revert this file to version from main branch before getting approval.

Copy link
Contributor

Choose a reason for hiding this comment

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

Agree :)

@kyma-bot kyma-bot added the area/documentation Issues or PRs related to documentation label Nov 20, 2023
muralov
muralov previously approved these changes Nov 24, 2023
@kyma-bot kyma-bot added the lgtm Looks good to me! label Nov 24, 2023
@kyma-bot kyma-bot removed the lgtm Looks good to me! label Nov 24, 2023
IwonaLanger
IwonaLanger previously approved these changes Nov 24, 2023
@kyma-bot kyma-bot added the lgtm Looks good to me! label Nov 24, 2023
@muralov muralov added area/eventing Issues or PRs related to eventing and removed do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. labels Nov 24, 2023
@kyma-bot kyma-bot removed the lgtm Looks good to me! label Nov 24, 2023
@kyma-bot kyma-bot added the lgtm Looks good to me! label Nov 27, 2023
@muralov muralov requested a review from NHingerl November 27, 2023 09:52
@kyma-bot kyma-bot merged commit 692195c into kyma-project:main Nov 27, 2023
6 checks passed
@mfaizanse mfaizanse deleted the move_code_part_6 branch December 11, 2023 09:18
friedrichwilken pushed a commit to friedrichwilken/eventing-manager that referenced this pull request Dec 14, 2023
* Moved api pkg from eventing-controller

* updated samples

* Fix failing tests

* Delete external CRD folder

* Revert 02-configuration.md doc

---------

Co-authored-by: Mansur Uralov <[email protected]>
friedrichwilken pushed a commit to friedrichwilken/eventing-manager that referenced this pull request Dec 14, 2023
* Moved api pkg from eventing-controller

* updated samples

* Fix failing tests

* Delete external CRD folder

* Revert 02-configuration.md doc

---------

Co-authored-by: Mansur Uralov <[email protected]>
friedrichwilken pushed a commit to friedrichwilken/eventing-manager that referenced this pull request Dec 14, 2023
* Moved api pkg from eventing-controller

* updated samples

* Fix failing tests

* Delete external CRD folder

* Revert 02-configuration.md doc

---------

Co-authored-by: Mansur Uralov <[email protected]>
friedrichwilken pushed a commit to friedrichwilken/eventing-manager that referenced this pull request Dec 14, 2023
* Moved api pkg from eventing-controller

* updated samples

* Fix failing tests

* Delete external CRD folder

* Revert 02-configuration.md doc

---------

Co-authored-by: Mansur Uralov <[email protected]>
kyma-bot pushed a commit that referenced this pull request Dec 14, 2023
* Moved api pkg from eventing-controller (#260)

* Moved api pkg from eventing-controller

* updated samples

* Fix failing tests

* Delete external CRD folder

* Revert 02-configuration.md doc

---------

Co-authored-by: Mansur Uralov <[email protected]>

* Cache runtime objects only created by the EventingManager (#273)

* Cache runtime objects only created by the EventingManager

* Update test case name

* Semantic versioning in security scan config (#319)

* check_image.sh

* check_image.sh

* release.sh

* Remove `.version` file check in release

* Call retry function

* Refactoring

* Verify status

* Check tags in security-scan-config.yaml

* Check on both place

* Collect latest run related data

* Checkout code (#323)

* Release input variable for tag checking (#324)

* Release input variable for tag checking (#326)

* Release input variable for tag checking

* Remove whitespace

* Remove white space at rc-tag

* Initial Wait Time (#327)

* Initial Wait Time

* Parameters

* Remove first check

* Run github action workflows on release-branches. (#330)

* tidy go.mod

* bump golangcli-lint vesion

* bump release version

---------

Co-authored-by: Muhammad Faizan <[email protected]>
Co-authored-by: Mansur Uralov <[email protected]>
Co-authored-by: Marco Bebway <[email protected]>
Co-authored-by: Tibor Kiss <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Issues or PRs related to documentation area/eventing Issues or PRs related to eventing 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.

Move code from eventing-controller to eventing-manager
5 participants