terascope / teraslice

Scalable data processing pipelines in JavaScript
https://terascope.github.io/teraslice/
Apache License 2.0
50 stars 13 forks source link

Refactor promMetrics to require check for its existence #3625

Closed busma13 closed 4 months ago

busma13 commented 4 months ago

We are going to remove the promMetricsProxy from terafoundation and instead check for the existence of promMetrics anywhere the API is called. We realized assets with promMetrics calls would need another proxy in the job-components core to make them compatible with older versions of teraslice. It seems much better to just simplify things and check for the existence of the API anytime we use it.

We will add a helper function into job-components to make the check a little cleaner.

We need to update the following list of asset releases to indicate that they should not be used:

godber commented 4 months ago

This PR implements the first step: https://github.com/terascope/teraslice/pull/3629

Next we have to go update all the assets to use this. When those are done, we need to come back here and remove the proxy in terafoundation. If we do it in the wrong order e2e tests faile.

busma13 commented 4 months ago

Caution: this asset is only compatible with teraslice v1.4.0 through v1.7.1

sotojn commented 4 months ago

*Do Not Use*

Reason: this version of the asset is incompatible with teraslice versions older thanv1.4.0 and and version greater than or equal to v1.7.2. This was because of a breaking change when implementing prom metrics.

godber commented 4 months ago

Do Not Use This Release: Use release X or Newer

NOTE: This release of the asset is only compatible with Teraslice versions between v1.4.0 and v1.7.1. This was because of a breaking change when implementing prom metrics.

godber commented 4 months ago

This and all the related work is completed.