Skip to content

fix: blacklist uts by transformation id#6832

Merged
fracasula merged 7 commits into
release/1.72.xfrom
fix.blaclistTransformation
Mar 31, 2026
Merged

fix: blacklist uts by transformation id#6832
fracasula merged 7 commits into
release/1.72.xfrom
fix.blaclistTransformation

Conversation

@fracasula

@fracasula fracasula commented Mar 31, 2026

Copy link
Copy Markdown
Collaborator

Description

Blacklisting transformations that do weird things like setting a duration_ms field, which will never be the same between the two transformers:

def transformEvent(event, metadata):
    start_time = time.time()
    # ...
    end_time = time.time()
    duration_ms = (end_time - start_time) * 1000
    event['duration_ms'] = duration_ms
    return event

Linear Ticket

< Fixes PIPE-2879 >

Security

  • The code changed/added as part of this pull request won't create any security issues with how the software is being used.

@fracasula fracasula marked this pull request as ready for review March 31, 2026 15:08
@codecov

codecov Bot commented Mar 31, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 66.66667% with 21 lines in your changes missing coverage. Please review.
✅ Project coverage is 79.66%. Comparing base (bb37aa4) to head (85be566).
⚠️ Report is 1 commits behind head on release/1.72.x.

Files with missing lines Patch % Lines
processor/processor.go 63.15% 18 Missing and 3 partials ⚠️
Additional details and impacted files
@@                Coverage Diff                 @@
##           release/1.72.x    #6832      +/-   ##
==================================================
- Coverage           80.20%   79.66%   -0.54%     
==================================================
  Files                 565      565              
  Lines               63376    63408      +32     
==================================================
- Hits                50829    50517     -312     
- Misses               9569     9885     +316     
- Partials             2978     3006      +28     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@fracasula fracasula enabled auto-merge (squash) March 31, 2026 15:54
@fracasula fracasula merged commit a1d9613 into release/1.72.x Mar 31, 2026
71 checks passed
@fracasula fracasula deleted the fix.blaclistTransformation branch March 31, 2026 17:33
fracasula pushed a commit that referenced this pull request Apr 1, 2026
🤖 I have created a release *beep* *boop*
---


##
[1.72.1](v1.72.0...v1.72.1)
(2026-03-31)


### Bug Fixes

* blacklist uts by transformation id
([#6832](#6832))
([a1d9613](a1d9613))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: rudderstack-github-actions[bot] <236995729+rudderstack-github-actions[bot]@users.noreply.github.com>
fracasula added a commit that referenced this pull request Apr 1, 2026
# Description

Syncing patch release v1.72.1 to main branch

**↓↓ Please review and edit commit overrides before merging ↓↓**

BEGIN_COMMIT_OVERRIDE
fix: blacklist uts by transformation id (#6832)
END_COMMIT_OVERRIDE

---------

Co-authored-by: Francesco Casula <fracasula@users.noreply.github.com>
Co-authored-by: rudderstack-github-actions[bot] <236995729+rudderstack-github-actions[bot]@users.noreply.github.com>
itsmihir pushed a commit that referenced this pull request Apr 13, 2026
🤖 I have created a release *beep* *boop*
---


##
[1.73.0-rc.1](v1.72.0...v1.73.0-rc.1)
(2026-04-13)


### Bug Fixes

* blacklist uts by transformation id
([#6832](#6832))
([a78fa93](a78fa93))
* **warehouse:** intermittent failures in bigquery integration test
partition listing
([#6862](#6862))
([f0bdc52](f0bdc52))
* **warehouse:** intermittent failures in bigquery integration test
schema verification
([#6858](#6858))
([8244315](8244315))


### Miscellaneous

* add dependabot cooldown
([#6842](#6842))
([8a96763](8a96763))
* apply security best practices from step security
([#6746](#6746))
([ec4390d](ec4390d))
* change release to 1.73.0
([370bbdb](370bbdb))
* **deps:** bump actions/checkout from 4.3.0 to 6.0.2
([#6848](#6848))
([f393a5f](f393a5f))
* **deps:** bump actions/create-github-app-token from 2.1.4 to 3.0.0
([#6845](#6845))
([7b5bd0f](7b5bd0f))
* **deps:** bump actions/download-artifact from 4.3.0 to 8.0.1
([#6849](#6849))
([3d0c5f5](3d0c5f5))
* **deps:** bump actions/setup-go from 5.5.0 to 6.4.0
([#6831](#6831))
([f4d7c78](f4d7c78))
* **deps:** bump actions/stale from 9.1.0 to 10.2.0
([#6805](#6805))
([fe697e3](fe697e3))
* **deps:** bump actions/upload-artifact from 4.6.2 to 7.0.0
([#6847](#6847))
([8cc34b8](8cc34b8))
* **deps:** bump codecov/codecov-action from 5.5.1 to 6.0.0
([#6830](#6830))
([667d363](667d363))
* **deps:** bump docker/login-action from 3.4.0 to 4.0.0
([#6808](#6808))
([fdd16b5](fdd16b5))
* **deps:** bump docker/metadata-action from 5.9.0 to 6.0.0
([#6851](#6851))
([766203b](766203b))
* **deps:** bump docker/setup-buildx-action from 3.11.1 to 4.0.0
([#6806](#6806))
([1ac9e51](1ac9e51))
* **deps:** bump github.com/evanphx/json-patch from 0.5.2 to
5.9.11+incompatible
([#6853](#6853))
([8526e45](8526e45))
* **deps:** bump go dependencies
([#6816](#6816))
([5630ae4](5630ae4))
* **deps:** bump slackapi/slack-github-action from 1.26.0 to 3.0.1
([#6846](#6846))
([b16d1f7](b16d1f7))
* **deps:** bump the go-deps group across 1 directory with 8 updates
([#6859](#6859))
([29c7c59](29c7c59))
* dispatch release events to operator/devops after image builds complete
([#6827](#6827))
([c9c0dde](c9c0dde))
* remove rsources_failed_keys migration and unused code
([#6837](#6837))
([9460873](9460873))
* **router:** kafka destination performance improvement
([#6836](#6836))
([90cbb45](90cbb45))
* sync release v1.72.0 to main branch
([#6829](#6829))
([cd13f36](cd13f36))
* update openfaas-flask ecr repo
([#6828](#6828))
([3aa2485](3aa2485))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: rudderstack-github-actions[bot] <236995729+rudderstack-github-actions[bot]@users.noreply.github.com>
itsmihir pushed a commit that referenced this pull request Apr 14, 2026
🤖 I have created a release *beep* *boop*
---


##
[1.73.0](v1.72.0...v1.73.0)
(2026-04-13)


### Bug Fixes

* blacklist uts by transformation id
([#6832](#6832))
([a78fa93](a78fa93))
* **warehouse:** intermittent failures in bigquery integration test
partition listing
([#6862](#6862))
([f0bdc52](f0bdc52))
* **warehouse:** intermittent failures in bigquery integration test
schema verification
([#6858](#6858))
([8244315](8244315))


### Miscellaneous

* add dependabot cooldown
([#6842](#6842))
([8a96763](8a96763))
* apply security best practices from step security
([#6746](#6746))
([ec4390d](ec4390d))
* change release to 1.73.0
([370bbdb](370bbdb))
* **deps:** bump actions/checkout from 4.3.0 to 6.0.2
([#6848](#6848))
([f393a5f](f393a5f))
* **deps:** bump actions/create-github-app-token from 2.1.4 to 3.0.0
([#6845](#6845))
([7b5bd0f](7b5bd0f))
* **deps:** bump actions/download-artifact from 4.3.0 to 8.0.1
([#6849](#6849))
([3d0c5f5](3d0c5f5))
* **deps:** bump actions/setup-go from 5.5.0 to 6.4.0
([#6831](#6831))
([f4d7c78](f4d7c78))
* **deps:** bump actions/stale from 9.1.0 to 10.2.0
([#6805](#6805))
([fe697e3](fe697e3))
* **deps:** bump actions/upload-artifact from 4.6.2 to 7.0.0
([#6847](#6847))
([8cc34b8](8cc34b8))
* **deps:** bump codecov/codecov-action from 5.5.1 to 6.0.0
([#6830](#6830))
([667d363](667d363))
* **deps:** bump docker/login-action from 3.4.0 to 4.0.0
([#6808](#6808))
([fdd16b5](fdd16b5))
* **deps:** bump docker/metadata-action from 5.9.0 to 6.0.0
([#6851](#6851))
([766203b](766203b))
* **deps:** bump docker/setup-buildx-action from 3.11.1 to 4.0.0
([#6806](#6806))
([1ac9e51](1ac9e51))
* **deps:** bump github.com/evanphx/json-patch from 0.5.2 to
5.9.11+incompatible
([#6853](#6853))
([8526e45](8526e45))
* **deps:** bump go dependencies
([#6816](#6816))
([5630ae4](5630ae4))
* **deps:** bump slackapi/slack-github-action from 1.26.0 to 3.0.1
([#6846](#6846))
([b16d1f7](b16d1f7))
* **deps:** bump the go-deps group across 1 directory with 8 updates
([#6859](#6859))
([29c7c59](29c7c59))
* dispatch release events to operator/devops after image builds complete
([#6827](#6827))
([c9c0dde](c9c0dde))
* remove rsources_failed_keys migration and unused code
([#6837](#6837))
([9460873](9460873))
* **router:** kafka destination performance improvement
([#6836](#6836))
([90cbb45](90cbb45))
* sync release v1.72.0 to main branch
([#6829](#6829))
([cd13f36](cd13f36))
* update openfaas-flask ecr repo
([#6828](#6828))
([3aa2485](3aa2485))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: rudderstack-github-actions[bot] <236995729+rudderstack-github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants