[pull] master from jaegertracing:master #176

pull · 2021-09-29T14:17:39Z

See Commits and Changes for more details.

Can you help keep this open source service alive? 💖 Please sponsor : )

Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>

codecov-commenter · 2021-09-29T15:20:26Z

Codecov Report

Merging #176 (330cb0d) into master (6196f42) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #176   +/-   ##
=======================================
  Coverage   87.72%   87.72%           
=======================================
  Files          93       93           
  Lines        5864     5866    +2     
=======================================
+ Hits         5144     5146    +2     
  Misses        553      553           
  Partials      167      167

Impacted Files	Coverage Δ
pkg/apis/jaegertracing/v1/jaeger_types.go	`87.50% <ø> (ø)`
pkg/deployment/all_in_one.go	`100.00% <ø> (ø)`
pkg/ingress/query.go	`100.00% <100.00%> (ø)`

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6196f42...330cb0d. Read the comment docs.

Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>

* Align ES index cleaner test witch changes in 1.26 In version 1.26 the index cleaner uses creation date from index metadata and not the date from index name so we cannot use span with old start date to create indices. Signed-off-by: Pavol Loffay <p.loffay@gmail.com> Co-authored-by: Ruben Vargas <ruben.vp8510@gmail.com>

Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>

* Avoid touching jaeger deps on deployment/ns controller Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com> * Add tests for namespace and deployment controllers Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>

Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>

Signed-off-by: Israel Blancas <iblancas@redhat.com> Co-authored-by: Ruben Vargas <ruben.vp8510@gmail.com>

) * Allow sidecar injection for query pod from other Jaeger instances Signed-off-by: Pavol Loffay <p.loffay@gmail.com> * Fix other if Signed-off-by: Pavol Loffay <p.loffay@gmail.com>

Signed-off-by: Marco Freyre <marco.fz85@gmail.com>

Signed-off-by: Israel Blancas <iblancas@redhat.com> Co-authored-by: Ruben Vargas <ruben.vp8510@gmail.com>

Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>

* Add E2E testing for ES Rollover feature Signed-off-by: Israel Blancas <iblancas@redhat.com> * Add E2E testing for ES Rollover feature Signed-off-by: Israel Blancas <iblancas@redhat.com> * Apply changes requested in CR Signed-off-by: Israel Blancas <iblancas@redhat.com> * Fix security issue Signed-off-by: Israel Blancas <iblancas@redhat.com> * Ensure the data is totally stored before finishing the program Signed-off-by: Israel Blancas <iblancas@redhat.com> * Fix condition check Signed-off-by: Israel Blancas <iblancas@redhat.com> * Ensure the spans are received Signed-off-by: Israel Blancas <iblancas@redhat.com> * Apply new reporter to other tests Signed-off-by: Israel Blancas <iblancas@redhat.com> * Fix ES Rollover E2E test Signed-off-by: Israel Blancas <iblancas@redhat.com> Co-authored-by: Ruben Vargas <ruben.vp8510@gmail.com>

Signed-off-by: Israel Blancas <iblancas@redhat.com>

#1584) * Fix gRPC flags for OpenShift Signed-off-by: Martin Schimandl <martin.schimandl@gmail.com> * Add new TLS testcase Signed-off-by: Martin Schimandl <martin.schimandl@gmail.com> * Add testcase for explicit reporter.grpc.host-port on OpenShift Signed-off-by: Martin Schimandl <martin.schimandl@gmail.com> Co-authored-by: Ruben Vargas <ruben.vp8510@gmail.com>

Signed-off-by: Israel Blancas <iblancas@redhat.com>

…space (#1583) * Change default OpenShift query ingress SAR to pods in the jaeger namespace Signed-off-by: Pavol Loffay <p.loffay@gmail.com> * fmt Signed-off-by: Pavol Loffay <p.loffay@gmail.com> * Fix Signed-off-by: Pavol Loffay <p.loffay@gmail.com> * Fix Signed-off-by: Pavol Loffay <p.loffay@gmail.com> * Fix Signed-off-by: Pavol Loffay <p.loffay@gmail.com> * Fix Signed-off-by: Pavol Loffay <p.loffay@gmail.com> * Fix Signed-off-by: Pavol Loffay <p.loffay@gmail.com> * Fix Signed-off-by: Pavol Loffay <p.loffay@gmail.com> * Add more tests Signed-off-by: Pavol Loffay <p.loffay@gmail.com>

* First approach Signed-off-by: Israel Blancas <iblancas@redhat.com> * Remove not needed files Signed-off-by: Israel Blancas <iblancas@redhat.com> * Autogenerate the test files Signed-off-by: Israel Blancas <iblancas@redhat.com> * Add documentation Signed-off-by: Israel Blancas <iblancas@redhat.com> * Fix gomplate installation Signed-off-by: Israel Blancas <iblancas@redhat.com> * Address comments from code review Signed-off-by: Israel Blancas <iblancas@redhat.com>

* Fix test files permissions and endlines Signed-off-by: Israel Blancas <iblancas@redhat.com> * Fix es-rollover endline Signed-off-by: Israel Blancas <iblancas@redhat.com> Co-authored-by: Ruben Vargas <ruben.vp8510@gmail.com>

* Migrate generate E2E test to use KUTTL Signed-off-by: Israel Blancas <iblancas@redhat.com> * Fix gomplate installation Signed-off-by: Israel Blancas <iblancas@redhat.com> * Remove not needed sleep Signed-off-by: Israel Blancas <iblancas@redhat.com> * Apply changes requested in #1576 Signed-off-by: Israel Blancas <iblancas@redhat.com> * Improve tests robustness Signed-off-by: Israel Blancas <iblancas@redhat.com> * Apply changes requested in #1576 Signed-off-by: Israel Blancas <iblancas@redhat.com>

Signed-off-by: Israel Blancas <iblancas@redhat.com>

* Migrate examples 1 tests to KUTTL Signed-off-by: Israel Blancas <iblancas@redhat.com> * Apply changes requested in #1576 Signed-off-by: Israel Blancas <iblancas@redhat.com> Co-authored-by: Ruben Vargas <ruben.vp8510@gmail.com>

Bumps [actions/checkout](https://github.com/actions/checkout) from 2.3.4 to 2.3.5. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@v2.3.4...v2.3.5) --- updated-dependencies: - dependency-name: actions/checkout dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Ruben Vargas <ruben.vp8510@gmail.com>

Signed-off-by: Israel Blancas <iblancas@redhat.com> Co-authored-by: Ruben Vargas <ruben.vp8510@gmail.com>

* Migrate cassandra tests to kuttl Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com> * Add cassandra spark test Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com> * Wait for schema job Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com> * Add line break at the end of assert file Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com> * Rename cassandra_spark to cassandra-spark Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com> * Make install step as a template Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com> * Separate jaeger creation from cassandra instance creation Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>

Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>

Signed-off-by: Israel Blancas <iblancas@redhat.com>

Some checks done previously are not really important but they are making the test fail randomly. Signed-off-by: Israel Blancas <iblancas@redhat.com> Co-authored-by: Ruben Vargas <ruben.vp8510@gmail.com>

Signed-off-by: Robert Kukura <rkukura@redhat.com>

Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>

Signed-off-by: Robert Kukura <rkukura@redhat.com>

…1678) Signed-off-by: Mihai Matache <mihai.matache@sap.com>

* Split the E2E tests in suites. Closes #1595 #1575 #1579 Signed-off-by: Israel Blancas <iblancas@redhat.com> * Fix issue in cassandra test Signed-off-by: Israel Blancas <iblancas@redhat.com> Co-authored-by: Ruben Vargas <ruben.vp8510@gmail.com>

Bumps [github.com/spf13/cobra](https://github.com/spf13/cobra) from 1.1.1 to 1.2.1. - [Release notes](https://github.com/spf13/cobra/releases) - [Changelog](https://github.com/spf13/cobra/blob/master/CHANGELOG.md) - [Commits](spf13/cobra@v1.1.1...v1.2.1) --- updated-dependencies: - dependency-name: github.com/spf13/cobra dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Ruben Vargas <ruben.vp8510@gmail.com>

Bumps [github.com/sirupsen/logrus](https://github.com/sirupsen/logrus) from 1.7.0 to 1.8.1. - [Release notes](https://github.com/sirupsen/logrus/releases) - [Changelog](https://github.com/sirupsen/logrus/blob/master/CHANGELOG.md) - [Commits](sirupsen/logrus@v1.7.0...v1.8.1) --- updated-dependencies: - dependency-name: github.com/sirupsen/logrus dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Ruben Vargas <ruben.vp8510@gmail.com>

Bumps [github.com/spf13/cobra](https://github.com/spf13/cobra) from 1.2.1 to 1.3.0. - [Release notes](https://github.com/spf13/cobra/releases) - [Changelog](https://github.com/spf13/cobra/blob/master/CHANGELOG.md) - [Commits](spf13/cobra@v1.2.1...v1.3.0) --- updated-dependencies: - dependency-name: github.com/spf13/cobra dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Signed-off-by: Abhijit Mane <abhijman@in.ibm.com> Co-authored-by: Ruben Vargas <ruben.vp8510@gmail.com>

Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>

Signed-off-by: Pavol Loffay <p.loffay@gmail.com>

* Fix leasing permisions Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com> * Fix linting Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>

Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>

* Avoid calling k8s api for each resource kind on the cluster Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com> * Update autodetect tests Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com> * Use map to filter groups Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com> * Simplify the group check Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com> * Increase coverage for autodetect Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>

Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>

Signed-off-by: Robert Kukura <rkukura@redhat.com>

Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>

* Add missing E2E sidecar tests Signed-off-by: Israel Blancas <iblancas@redhat.com> * Typo Signed-off-by: Israel Blancas <iblancas@redhat.com> Co-authored-by: Ruben Vargas <ruben.vp8510@gmail.com>

* #1729: improve robustness of wait-cronjob Signed-off-by: Israel Blancas <iblancas@redhat.com> * #1729: remove superfluous return Signed-off-by: Israel Blancas <iblancas@redhat.com>

…1708) The jaeger-elasticsearch certificate will be managed by ES and will not contain Jaeger instance name. The name will be jaeger-<es-instance-name>. Signed-off-by: Pavol Loffay <p.loffay@gmail.com>

Closes #1641 Signed-off-by: Benedikt Bongartz <bongartz@klimlive.de>

Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>

* #1726: split each E2E test suite in a different workflow to make them easier to reboot Signed-off-by: Israel Blancas <iblancas@redhat.com> * #1726: add sidecar tests Signed-off-by: Israel Blancas <iblancas@redhat.com> * #1726: remove duplicated file Signed-off-by: Israel Blancas <iblancas@redhat.com>

Signed-off-by: Benedikt Bongartz <bongartz@klimlive.de>

Bumps [actions/setup-go](https://github.com/actions/setup-go) from 2.1.4 to 2.2.0. - [Release notes](https://github.com/actions/setup-go/releases) - [Commits](actions/setup-go@v2.1.4...v2.2.0) --- updated-dependencies: - dependency-name: actions/setup-go dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Add ingressClassName field to query ingress (#1557)

330cb0d

Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>

pull bot added the ⤵️ pull label Sep 29, 2021

rubenvp8510 and others added 27 commits September 29, 2021 17:29

Set version of the nginx ingress controller to 1.0.1 (#1563)

6457733

Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>

Preparing release v1.26.0 (#1552)

dd42dab

Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>

Use github token for release (#1566)

68cc6ee

Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>

Avoid touching jaeger deps on deployment/ns controller (#1529)

5614ac6

* Avoid touching jaeger deps on deployment/ns controller Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com> * Add tests for namespace and deployment controllers Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>

Fix operatorhub paths (#1567)

175eeb6

Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>

Decrease the time of the container image build (#1545)

c1e6ee1

Signed-off-by: Israel Blancas <iblancas@redhat.com> Co-authored-by: Ruben Vargas <ruben.vp8510@gmail.com>

Allow sidecar injection for query pod from other Jaeger instances (#1569

a95ea91

) * Allow sidecar injection for query pod from other Jaeger instances Signed-off-by: Pavol Loffay <p.loffay@gmail.com> * Fix other if Signed-off-by: Pavol Loffay <p.loffay@gmail.com>

bump kuttl version to 0.11.1 (#1570)

59bb2ce

Signed-off-by: Marco Freyre <marco.fz85@gmail.com>

Add E2E testing when the collector is exposed (#1548)

ccfce8e

Signed-off-by: Israel Blancas <iblancas@redhat.com> Co-authored-by: Ruben Vargas <ruben.vp8510@gmail.com>

Preparing release v1.27.0 (#1577)

93eb3c4

Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>

Increase the timeout for the test. Closes #1590 (#1591)

d9e6533

Signed-off-by: Israel Blancas <iblancas@redhat.com>

Enable verbosity in the makefile easily (#1547)

6584f4a

Signed-off-by: Israel Blancas <iblancas@redhat.com>

Fix test files permissions and endlines (#1578)

b4c8653

* Fix test files permissions and endlines Signed-off-by: Israel Blancas <iblancas@redhat.com> * Fix es-rollover endline Signed-off-by: Israel Blancas <iblancas@redhat.com> Co-authored-by: Ruben Vargas <ruben.vp8510@gmail.com>

Add git hook to format and lint the code (#1554)

f208d0f

Signed-off-by: Israel Blancas <iblancas@redhat.com>

Migrate examples 1 tests to KUTTL (#1589)

4bf0a28

* Migrate examples 1 tests to KUTTL Signed-off-by: Israel Blancas <iblancas@redhat.com> * Apply changes requested in #1576 Signed-off-by: Israel Blancas <iblancas@redhat.com> Co-authored-by: Ruben Vargas <ruben.vp8510@gmail.com>

Migrate istio E2E test to KUTTL (#1582)

a7883a6

Signed-off-by: Israel Blancas <iblancas@redhat.com> Co-authored-by: Ruben Vargas <ruben.vp8510@gmail.com>

Fix asserting number of cassandra replicas (#1599)

39bdf76

Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>

Improve Istio E2E test to avoid errors (#1602)

b61576a

Signed-off-by: Israel Blancas <iblancas@redhat.com>

Simplify ES Rollover E2E test. (#1600)

7776f0b

Some checks done previously are not really important but they are making the test fail randomly. Signed-off-by: Israel Blancas <iblancas@redhat.com> Co-authored-by: Ruben Vargas <ruben.vp8510@gmail.com>

Bob Kukura and others added 28 commits December 13, 2021 18:17

Fix dry-run arg in Makefile (#1670)

35ac31f

Signed-off-by: Robert Kukura <rkukura@redhat.com>

Fix OLM metadata (#1669)

cb609cb

Signed-off-by: Robert Kukura <rkukura@redhat.com>

Register oschema for openshift resources (#1673)

3626129

Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>

Preparing release v1.29.1 (#1674)

1e9c2ad

Signed-off-by: Robert Kukura <rkukura@redhat.com>

util.Truncate add the values to the truncated after the excess is 0 (#…

c30ca9d

…1678) Signed-off-by: Mihai Matache <mihai.matache@sap.com>

Add support for IBM Power (ppc64le) arch (#1672)

01795af

Signed-off-by: Abhijit Mane <abhijman@in.ibm.com> Co-authored-by: Ruben Vargas <ruben.vp8510@gmail.com>

Register ES types (#1688)

e9d1a5e

Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>

Change spark-dependencies image to GHCR (#1701)

ac6d50e

Signed-off-by: Pavol Loffay <p.loffay@gmail.com>

Add leases permissions to manifest. (#1704)

a6bfbc5

* Fix leasing permisions Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com> * Fix linting Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>

Restore default metrics port to avoid breaking helm (#1703)

e2b1ccc

Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>

Add permissions for imagestreams (#1714)

57d3d92

Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>

First call of autodetect should be synchronous (#1713)

343c4f2

Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>

Add CR Metrics (#1706)

82ee678

Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>

Only expose the query-http[s] port in the OpenShift route (#1719)

f677238

Signed-off-by: Robert Kukura <rkukura@redhat.com>

Preparing release v1.30.0 (#1711)

edd66c7

Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>

Add missing E2E sidecar tests (#1718)

4ba0172

* Add missing E2E sidecar tests Signed-off-by: Israel Blancas <iblancas@redhat.com> * Typo Signed-off-by: Israel Blancas <iblancas@redhat.com> Co-authored-by: Ruben Vargas <ruben.vp8510@gmail.com>

#1729: improve robustness of wait-cronjob (#1730)

f990030

* #1729: improve robustness of wait-cronjob Signed-off-by: Israel Blancas <iblancas@redhat.com> * #1729: remove superfluous return Signed-off-by: Israel Blancas <iblancas@redhat.com>

Use Elasticsearch provisioning from OpenShift Elasticsearch operator (#…

eb57675

…1708) The jaeger-elasticsearch certificate will be managed by ES and will not contain Jaeger instance name. The name will be jaeger-<es-instance-name>. Signed-off-by: Pavol Loffay <p.loffay@gmail.com>

get rid of deprication warning when installing dependencies (#1734)

b415094

Closes #1641 Signed-off-by: Benedikt Bongartz <bongartz@klimlive.de>

Add ES autoprovisioning CR metric (#1728)

69f88f5

Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>

fix panic caused by an invalid type assertion (#1738)

06f6880

Signed-off-by: Benedikt Bongartz <bongartz@klimlive.de>

rubenvp8510 deleted the branch Vinta-IAAS-Labs:master February 11, 2022 01:45

rubenvp8510 deleted the master branch February 11, 2022 01:45

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[pull] master from jaegertracing:master #176

[pull] master from jaegertracing:master #176

pull bot commented Sep 29, 2021 •

edited

codecov-commenter commented Sep 29, 2021

[pull] master from jaegertracing:master #176

Are you sure you want to change the base?

[pull] master from jaegertracing:master #176

Conversation

pull bot commented Sep 29, 2021 • edited

codecov-commenter commented Sep 29, 2021

Codecov Report

pull bot commented Sep 29, 2021 •

edited