Skip to content

Revert fluentbit metric bump telemetry manager image#18002

Closed
hisarbalik wants to merge 39 commits intokyma-project:release-2.17from
hisarbalik:revert-fluentbit-metric-bump-telemetry-manager-image
Closed

Revert fluentbit metric bump telemetry manager image#18002
hisarbalik wants to merge 39 commits intokyma-project:release-2.17from
hisarbalik:revert-fluentbit-metric-bump-telemetry-manager-image

Conversation

@hisarbalik
Copy link
Contributor

Description

Changes proposed in this pull request:

  • Fluent-bit metric exposure changes reverted

Related issue(s)
#17976
kyma-project/telemetry-manager#340

Sawthis and others added 30 commits July 31, 2023 16:31
* add api-gateway test logic for node

* adjust tests to check the runtime

* add python39 test

* uncomment main

* adjust pod selector

* add check if all labels exists

* adjust labels and error handling

* rename package
* Enable docsify

* Convert title to first level header

* Update getting started

* Update getting started

* API Gateway tutorials

* API Gateway tutorials

* Multiple sidebars

* Restore accidentally removed paragraph

* Fix links
* Small fixes in Quick Install docs

* Apply suggestions from code review

Co-authored-by: Natalia Sitko <80401180+nataliasitko@users.noreply.github.com>

---------

Co-authored-by: Natalia Sitko <80401180+nataliasitko@users.noreply.github.com>
* Add bash syntax highlighter

* Tab support

* Improve tabs handling
* Clean up API exposure sidebar

* Switch to keda-manager main repo

* Wider sidebar

* Revert sidebar changes
* Add _sidebar.md to Overview docs

* Add _sidebar.md files, consolidate some of the documents

* Update sidebar.md and adjust docs

* Bring Telemetry docs back

* Technical reference sidebar

* Add technical reference sidebar.md files

* Apply Iwona's suggestions from lg review
* Fix bug dispatching messages to old sink

* Update image

* Fix lint issues

* Update tests

* Fix naming

* Update interface docs
…project#17569)

* add retry for failing test

* retry added

* lint fixes

* add retry

* updated

* timeout fix

* retry added

* add retrty for failing k8sapply

* lint fixes

* duplicate functions removed

* missing value added
….6-alpine3.18 in application connector components (kyma-project#17914)

* docker(deps): bump kyma-project/prod/external/golang

Bumps kyma-project/prod/external/golang from 1.20.5-alpine3.18 to 1.20.6-alpine3.18.

---
updated-dependencies:
- dependency-name: kyma-project/prod/external/golang
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

* docker(deps): bump kyma-project/prod/external/golang

Bumps kyma-project/prod/external/golang from 1.20.5-alpine3.18 to 1.20.6-alpine3.18.

---
updated-dependencies:
- dependency-name: kyma-project/prod/external/golang
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

* docker(deps): bump kyma-project/prod/external/golang

Bumps kyma-project/prod/external/golang from 1.20.5-alpine3.18 to 1.20.6-alpine3.18.

---
updated-dependencies:
- dependency-name: kyma-project/prod/external/golang
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

* bumps app-con images to 17914

* bumps app-con tests golang images from 1.19.X to 1.20.6

* bumps app-conn test images

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* API Gateway 1.8.0

* Update docs

* Update apix-01-apirule.md

* Update apix-01-apirule.md

* Auto generated docu

* Add review note

* Update post-review

* Apply suggestions from code review

Co-authored-by: Natalia Sitko <80401180+nataliasitko@users.noreply.github.com>

* Generate table

* Apply suggestions from code review

Co-authored-by: Natalia Sitko <80401180+nataliasitko@users.noreply.github.com>

* Generate table

* Apply suggestions from code review

Co-authored-by: Natalia Sitko <80401180+nataliasitko@users.noreply.github.com>

* Generate table

* Update installation/resources/crds/api-gateway/apirules.gateway.crd.yaml

Co-authored-by: Natalia Sitko <80401180+nataliasitko@users.noreply.github.com>

* Generate table

---------

Co-authored-by: Natalia Sitko <80401180+nataliasitko@users.noreply.github.com>
* add testenv

* upgrade git2go

* bump image and update docs

* update dockerfiles

* remove trash

* update job init
* Add more sidebar.md files

* Change to relative links where possible

* Update link

* Update sidebars

* Apply review suggestions
* Fix the APIRule OwnerReferences equality check

* Update image
* introduce subscription status as a metric

* fix long lines

* enable new metric

* update documentation

* fix missing import

* add subscription-status metric for eventmesh backend

* fix long lines

* bump images

* reset subscription status metric on backend change

* add stream_name as label on the subscription_status-metric

* fix long lines

* fix according to review comments

* shorter lines
* Remove deprecated JaegerServiceEndpoint

* bump image
* Improve istio component tests

* Remove old step

* Refactor test

* Refactor tests

* Update tests/components/istio/README.md

Co-authored-by: Vladimir Videlov <vladimir.videlov@sap.com>

* Remove redundant function

* Update tests/components/istio/README.md

Co-authored-by: Natalia Sitko <80401180+nataliasitko@users.noreply.github.com>

---------

Co-authored-by: Vladimir Videlov <vladimir.videlov@sap.com>
Co-authored-by: Natalia Sitko <80401180+nataliasitko@users.noreply.github.com>
* Unify sample function in docs

* Update docs/02-get-started/02-deploy-expose-function.md

Co-authored-by: Małgorzata Świeca <malgorzata.swieca@sap.com>

---------

Co-authored-by: Małgorzata Świeca <malgorzata.swieca@sap.com>
* remove the tutorial

* fix links
halamix2 and others added 9 commits August 10, 2023 10:46
* Bump k8s-tools image

* revet prettify

* revet prettify
* Configuration for installation of Istio operator

* Add operator image version for scanning

* Use the struct for image scanners
* Preserve EventMesh precomputed hashes

* Revert unnecessary changes

* Improve test readability
* sync telemetry docs in module repo and kyma repo

* align more details

* revert change in resources file

* replace caution with note

* replace caution with note
* Update documentaion of Istio component

* Apply suggestions from code review

Co-authored-by: Natalia Sitko <80401180+nataliasitko@users.noreply.github.com>

---------

Co-authored-by: Natalia Sitko <80401180+nataliasitko@users.noreply.github.com>
* improve cli usage

* fixd

* imprve var name
* first steps

* try with python test function

* add simple check

* enable nodejs18 check and add ability to run test manually

* add 2 event checks

* improve assertion and do some renaming

* extract to methods

* small fixes

* uncomment nodejs16

* small fixes

---------

Co-authored-by: Marcin Dobrochowski <anoip@o2.pl>
@hisarbalik hisarbalik requested review from a team as code owners August 18, 2023 13:22
@kyma-bot kyma-bot added the do-not-merge/invalid-commit-message Indicates that a PR should not merge because it has an invalid commit message. label Aug 18, 2023
@kyma-bot
Copy link
Contributor

Keywords which can automatically close issues and at(@) or hashtag(#) mentions are not allowed in commit messages.

The list of commits with invalid commit messages:

Details

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.

@hisarbalik hisarbalik closed this Aug 18, 2023
@kyma-bot kyma-bot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Aug 18, 2023
@kyma-bot
Copy link
Contributor

@hisarbalik: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
pull-eventing-controller-unit-test a7f15f4 link true /test pull-eventing-controller-unit-test
pull-eventing-controller-lint a7f15f4 link true /test pull-eventing-controller-lint
pull-function-controller-unit-test a7f15f4 link true /test pull-function-controller-unit-test
pull-application-connector-gateway-test-build a7f15f4 link false /test pull-application-connector-gateway-test-build
pre-main-kyma-integration-k3d a7f15f4 link true /test pre-main-kyma-integration-k3d
pre-release-pr-image-guard a7f15f4 link true /test pre-release-pr-image-guard
pull-application-connector-mock-app-build a7f15f4 link false /test pull-application-connector-mock-app-build
pull-central-application-gateway-build a7f15f4 link true /test pull-central-application-gateway-build
pre-kyma-presubmit-scanner a7f15f4 link false /test pre-kyma-presubmit-scanner
pull-application-connector-connectivity-validator-test-build a7f15f4 link false /test pull-application-connector-connectivity-validator-test-build
pre-main-kyma-integration-k3d-telemetry a7f15f4 link true /test pre-main-kyma-integration-k3d-telemetry
pull-function-webhook-build a7f15f4 link true /test pull-function-webhook-build
pull-function-controller-build a7f15f4 link true /test pull-function-controller-build
pull-function-runtimes-nodejs-v16-build a7f15f4 link true /test pull-function-runtimes-nodejs-v16-build
pull-function-registry-gc-build a7f15f4 link false /test pull-function-registry-gc-build
pull-eventing-controller-build a7f15f4 link true /test pull-eventing-controller-build

Full PR test history. Your PR dashboard.

Details

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge/invalid-commit-message Indicates that a PR should not merge because it has an invalid commit message. 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.