Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge v1.20 into v1.x #1444

Merged
merged 1 commit into from
Sep 24, 2024
Merged

Conversation

mongodb-php-bot
Copy link
Collaborator

Merge new changes from v1.20 into v1.x.

Commits

Resolving conflicts

To resolve any conflicts, check out the temporary branch and run the following command:
git merge v1.x

Ignoring changes

To ignore from the remote branch, first reset the temporary branch to v1.x and manually merge using the `ours` merge strategy:
git reset --hard v1.x
git merge --strategy=ours v1.20

Then, push the temporary branch to upate the pull request.

…or logs (#1428)

* Track servers by host:port instead of serverConnectionId

The server's connection ID is unique within the context of a single server and is only reported by MongoDB 4.2+. It should not be used to uniquely identify a server.

This reverts recent changes from 50d1db7.

* Rename connectionId and add extra fields to EventCollector logs

This renames connectionId to server, to avoid ambiguity with serverConnectionId.

Additional fields have also been added to the output. Since PHPC 1.19, database name is available on all APM events.
@jmikola
Copy link
Member

jmikola commented Sep 23, 2024

Note: the CI failures here are related to #1427, which introduced flaky failures when disabling fail points.

@alcaeus
Copy link
Member

alcaeus commented Sep 24, 2024

@jmikola there were a few other flaky tests on evergreen. I kicked the jobs to restart them so we can merge this before releasing.

@alcaeus alcaeus removed the request for review from jmikola September 24, 2024 11:39
@mongodb-php-bot mongodb-php-bot merged commit 72d205f into v1.x Sep 24, 2024
58 checks passed
@mongodb-php-bot mongodb-php-bot deleted the merge-v1.20-into-v1.x-1727117340472 branch September 24, 2024 11:57
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