Skip to content

[v14] fix leaf SSH sessions not getting recorded#33102

Merged
capnspacehook merged 1 commit intobranch/v14from
capnspacehook/backport/v14/32163
Oct 9, 2023
Merged

[v14] fix leaf SSH sessions not getting recorded#33102
capnspacehook merged 1 commit intobranch/v14from
capnspacehook/backport/v14/32163

Conversation

@capnspacehook
Copy link
Copy Markdown
Contributor

Backport of #32163.

Not a clean backport, but the only new change is fixing withProxyRecordingMode's function signature in integration_test.go.

@capnspacehook capnspacehook changed the base branch from master to branch/v14 October 6, 2023 21:45
@capnspacehook capnspacehook force-pushed the capnspacehook/backport/v14/32163 branch from 64a7781 to 3ce0701 Compare October 9, 2023 18:38
* fix leaf SSH sessions not getting recorded

* add integration test

* address feedback, overhaul integration test

* make each test case use fresh clusters to fix failing case

* address feedback

* Apply suggestions from code review

Co-authored-by: rosstimothy <39066650+rosstimothy@users.noreply.github.com>

* fix integration test failures

---------

Co-authored-by: rosstimothy <39066650+rosstimothy@users.noreply.github.com>
@capnspacehook capnspacehook force-pushed the capnspacehook/backport/v14/32163 branch from 3ce0701 to 9ff3b98 Compare October 9, 2023 20:30
@capnspacehook capnspacehook added this pull request to the merge queue Oct 9, 2023
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Oct 9, 2023
@capnspacehook capnspacehook added this pull request to the merge queue Oct 9, 2023
Merged via the queue into branch/v14 with commit 27ce3e7 Oct 9, 2023
@capnspacehook capnspacehook deleted the capnspacehook/backport/v14/32163 branch October 9, 2023 23:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants