Skip to content

Conversation

@sampaiodiego
Copy link
Member

@sampaiodiego sampaiodiego commented Feb 12, 2025

The fix will be using a random id for each ddp-streamer instance so in a restart a new id will be created, causing the old connections being pruned by the existing code that prunes connections from non-existent instances.

Proposed changes (including videos or screenshots)

Issue(s)

ARCH-1453

Steps to test or reproduce

Further comments

@dionisio-bot
Copy link
Contributor

dionisio-bot bot commented Feb 12, 2025

Looks like this PR is ready to merge! 🎉
If you have any trouble, please check the PR guidelines

@changeset-bot
Copy link

changeset-bot bot commented Feb 12, 2025

⚠️ No Changeset found

Latest commit: 47f8014

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@codecov
Copy link

codecov bot commented Feb 12, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 59.44%. Comparing base (f89741d) to head (47f8014).
Report is 1 commits behind head on develop.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff            @@
##           develop   #35195   +/-   ##
========================================
  Coverage    59.44%   59.44%           
========================================
  Files         2827     2827           
  Lines        68163    68163           
  Branches     15123    15123           
========================================
  Hits         40517    40517           
  Misses       24990    24990           
  Partials      2656     2656           
Flag Coverage Δ
unit 75.69% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@sampaiodiego sampaiodiego marked this pull request as ready for review February 12, 2025 23:27
@sampaiodiego sampaiodiego requested a review from a team as a code owner February 12, 2025 23:27
@sampaiodiego sampaiodiego added this to the 7.4.0 milestone Feb 13, 2025
@sampaiodiego sampaiodiego added the stat: QA assured Means it has been tested and approved by a company insider label Feb 14, 2025
@dionisio-bot dionisio-bot bot added the stat: ready to merge PR tested and approved waiting for merge label Feb 14, 2025
@kodiakhq kodiakhq bot merged commit a1d3faa into develop Feb 14, 2025
47 of 48 checks passed
@kodiakhq kodiakhq bot deleted the fix-ddp-streamer-restart branch February 14, 2025 20:14
@sampaiodiego
Copy link
Member Author

/patch

@dionisio-bot
Copy link
Contributor

dionisio-bot bot commented Feb 14, 2025

Pull request #35220 added to Project: "Patch 7.3.2"

@sampaiodiego
Copy link
Member Author

/backport 7.2.3

@dionisio-bot
Copy link
Contributor

dionisio-bot bot commented Feb 17, 2025

Sorry, I couldn't do that backport because of conflicts. Could you please solve them?

you can do so by running the following commands:

git fetch
git checkout backport-7.2.3-35195
git cherry-pick a1d3faa0e2ee9c347e45e4e75a211983ba702820
// solve the conflict
git push

after that just run /backport 7.2.3 again

@sampaiodiego
Copy link
Member Author

/backport 7.2.3

@dionisio-bot
Copy link
Contributor

dionisio-bot bot commented Feb 17, 2025

Pull request #35241 added to Project: "Patch 7.2.3"

@sampaiodiego
Copy link
Member Author

/backport 7.1.3

@dionisio-bot
Copy link
Contributor

dionisio-bot bot commented Feb 17, 2025

Sorry, I couldn't do that backport because of conflicts. Could you please solve them?

you can do so by running the following commands:

git fetch
git checkout backport-7.1.3-35195
git cherry-pick a1d3faa0e2ee9c347e45e4e75a211983ba702820
// solve the conflict
git push

after that just run /backport 7.1.3 again

@sampaiodiego
Copy link
Member Author

/backport 7.1.3

@dionisio-bot
Copy link
Contributor

dionisio-bot bot commented Feb 17, 2025

Pull request #35242 added to Project: "Patch 7.1.3"

@sampaiodiego
Copy link
Member Author

/backport 7.0.7

@dionisio-bot
Copy link
Contributor

dionisio-bot bot commented Feb 17, 2025

Sorry, I couldn't do that backport because of conflicts. Could you please solve them?

you can do so by running the following commands:

git fetch
git checkout backport-7.0.7-35195
git cherry-pick a1d3faa0e2ee9c347e45e4e75a211983ba702820
// solve the conflict
git push

after that just run /backport 7.0.7 again

@sampaiodiego
Copy link
Member Author

/backport 7.0.7

@dionisio-bot
Copy link
Contributor

dionisio-bot bot commented Feb 17, 2025

Pull request #35243 added to Project: "Patch 7.0.7"

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

Labels

stat: QA assured Means it has been tested and approved by a company insider stat: ready to merge PR tested and approved waiting for merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants