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.

Sign up for GitHub

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

Jump to bottom

Remove deprecated reactor-based test code #482

Closed
kgiusti opened this issue May 18, 2022 · 0 comments · Fixed by #488
Closed

Remove deprecated reactor-based test code #482

kgiusti opened this issue May 18, 2022 · 0 comments · Fixed by #488
Assignees
@kgiusti
Milestone
2.1.0

Comments

@kgiusti
Copy link
Contributor

kgiusti commented May 18, 2022

reactor is deprecated, and there are now ASAN issues associated with its use that will not be fixed.

Refer to #469 (comment) for leak traceback
@kgiusti kgiusti self-assigned this May 18, 2022
@kgiusti kgiusti mentioned this issue May 18, 2022
Closed
@kgiusti kgiusti added this to the 2.1.0 milestone May 18, 2022
kgiusti added a commit to kgiusti/skupper-router that referenced this issue May 19, 2022
@kgiusti
Fixes skupperproject#482: replace reactor tests with proactor impleme…
82dc96a 
…ntation
@kgiusti kgiusti linked a pull request May 19, 2022 that will close this issue
Fixes #482: replace reactor tests with proactor implementation #488
Merged
kgiusti added a commit that referenced this issue May 19, 2022
@kgiusti
Fixes #482: replace reactor tests with proactor implementation (#488)
ff07362
kgiusti added a commit to kgiusti/skupper-router that referenced this issue May 19, 2022
@kgiusti
Fixes skupperproject#482: replace reactor tests with proactor impleme…
13bc19b 
…ntation (skupperproject#488)

(cherry picked from commit ff07362)
kgiusti added a commit that referenced this issue May 23, 2022
@kgiusti
Fixes #482: replace reactor tests with proactor implementation (#488) (…
56dd483 
…#491)

(cherry picked from commit ff07362)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kgiusti kgiusti

Labels
None yet
Projects
None yet
Milestone
2.1.0
Development

Successfully merging a pull request may close this issue.

Fixes #482: replace reactor tests with proactor implementation kgiusti/skupper-router
1 participant
@kgiusti