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

Eventing Manager Module Documentation #226

Merged
merged 11 commits into from
Nov 9, 2023

Conversation

grischperl
Copy link
Collaborator

@grischperl grischperl commented Nov 8, 2023

Description

Changes proposed in this pull request:

  • Documentation for contributors
  • Documentation for user
  • Updated README links
  • Automatically generate CRD documentation

Related issue(s)

#184

@kyma-bot
Copy link

kyma-bot commented Nov 8, 2023

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@kyma-bot kyma-bot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. area/documentation Issues or PRs related to documentation labels Nov 8, 2023
@grischperl grischperl linked an issue Nov 8, 2023 that may be closed by this pull request
12 tasks
@kyma-bot kyma-bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Nov 8, 2023
@grischperl grischperl marked this pull request as ready for review November 9, 2023 14:03
@grischperl grischperl requested review from a team as code owners November 9, 2023 14:03
@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 Nov 9, 2023
docs/contributor/development.md Outdated Show resolved Hide resolved
docs/contributor/development.md Outdated Show resolved Hide resolved
docs/contributor/governance.md Outdated Show resolved Hide resolved
docs/contributor/governance.md Outdated Show resolved Hide resolved
docs/contributor/governance.md Outdated Show resolved Hide resolved
docs/user/01-manager.md Outdated Show resolved Hide resolved
docs/user/01-manager.md Outdated Show resolved Hide resolved
docs/user/01-manager.md Outdated Show resolved Hide resolved
docs/user/02-configuration.md Outdated Show resolved Hide resolved
docs/user/README.md Outdated Show resolved Hide resolved
friedrichwilken
friedrichwilken previously approved these changes Nov 9, 2023
@kyma-bot kyma-bot added the lgtm Looks good to me! label Nov 9, 2023
docs/user/01-manager.md Outdated Show resolved Hide resolved
docs/user/01-manager.md Outdated Show resolved Hide resolved
docs/contributor/installation.md Outdated Show resolved Hide resolved
docs/contributor/installation.md Outdated Show resolved Hide resolved
docs/contributor/installation.md Outdated Show resolved Hide resolved
@kyma-bot kyma-bot added the lgtm Looks good to me! label Nov 9, 2023
@kyma-bot kyma-bot removed the lgtm Looks good to me! label Nov 9, 2023
@kyma-bot kyma-bot added the lgtm Looks good to me! label Nov 9, 2023
@kyma-bot kyma-bot merged commit fb65def into kyma-project:main Nov 9, 2023
9 checks passed
marcobebway pushed a commit to marcobebway/eventing-manager that referenced this pull request Nov 13, 2023
* Add the docs for contributor

* User documentation

* Add automatically generated CRD documentation

* Small improvements

* Replace link

* Apply review comments

* Add review comments

* Fix typo

* Add review comments
kyma-bot pushed a commit that referenced this pull request Nov 14, 2023
… replicas (#227)

* Ignore reconciling the EPP deployment if the only diff is in the spec replicas

* Fix tests

* Fix lint issues

* Set desired EPP deployment replicas to nil if there is not HPA

* Add integration test

* Ignore setting HPA

* fix equality tests for EPP

* moarcrap

* Update module docs (#230)

* Update module docs

* Further improvements

* Remove parallel test run (#224)

* Change spelling.

* Remove parallel execution from test `Update Eventing CR`.

* Remove all parallel running.

* Eventing Manager Module Documentation (#226)

* Add the docs for contributor

* User documentation

* Add automatically generated CRD documentation

* Small improvements

* Replace link

* Apply review comments

* Add review comments

* Fix typo

* Add review comments

* Bump golang.org/x/oauth2 from 0.13.0 to 0.14.0 (#232)

Bumps [golang.org/x/oauth2](https://github.com/golang/oauth2) from 0.13.0 to 0.14.0.
- [Commits](golang/oauth2@v0.13.0...v0.14.0)

---
updated-dependencies:
- dependency-name: golang.org/x/oauth2
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump github.com/vektra/mockery/v2 from 2.36.0 to 2.36.1 (#219)

Bumps [github.com/vektra/mockery/v2](https://github.com/vektra/mockery) from 2.36.0 to 2.36.1.
- [Release notes](https://github.com/vektra/mockery/releases)
- [Changelog](https://github.com/vektra/mockery/blob/master/docs/changelog.md)
- [Commits](vektra/mockery@v2.36.0...v2.36.1)

---
updated-dependencies:
- dependency-name: github.com/vektra/mockery/v2
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump github.com/onsi/gomega from 1.29.0 to 1.30.0 (#229)

Bumps [github.com/onsi/gomega](https://github.com/onsi/gomega) from 1.29.0 to 1.30.0.
- [Release notes](https://github.com/onsi/gomega/releases)
- [Changelog](https://github.com/onsi/gomega/blob/master/CHANGELOG.md)
- [Commits](onsi/gomega@v1.29.0...v1.30.0)

---
updated-dependencies:
- dependency-name: github.com/onsi/gomega
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Closing NATS connection on subscription manager shutdown and disable parallel reconciliations (#233)

* Closing NATS connection on subscription manager shutdown

* added logs

* set concurrent reconciliations to 0

* updated

* revert doc change

* fix tests by reverting most of the previous changes and introducing
predicates for all watched resources

* change predicates

introduce some logging

* skip on create

* ignore updates on cr /crb

* cleanup predicate code

* add more info to predicates

* change loglevel for predicates

* fix broken build

* more cleanup

* debug

* debug

* debug

* debug

* Address review comments

- Refactoring
- Add unit tests

* fix imports

* fix tests

* fix tests

* fix equality for envs

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: Korbinian Stoemmer <[email protected]>
Co-authored-by: Iwona Langer <[email protected]>
Co-authored-by: Friedrich <[email protected]>
Co-authored-by: Carina Kothe <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Muhammad Faizan <[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 lgtm Looks good to me! size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Official documentation approved by the TW
4 participants