Skip to content

My fix for pr99#100

Draft
AAYUSH2091 wants to merge 11 commits intoansible:develfrom
AAYUSH2091:my-fix-for-pr99
Draft

My fix for pr99#100
AAYUSH2091 wants to merge 11 commits intoansible:develfrom
AAYUSH2091:my-fix-for-pr99

Conversation

@AAYUSH2091
Copy link

@AAYUSH2091 AAYUSH2091 commented Dec 9, 2025

Description

  • Implemented end-to-end manager process lifecycle management, including PID tracking, graceful RPC shutdown, and force-kill fallback.
  • To prevent resource leaks (orphaned processes)
  • The action plugin now tracks spawned processes and triggers a cleanup

Type of Change

  • New feature (non-breaking change which adds functionality)

Self-Review Checklist

  • I have performed a self-review of my code
  • I have added relevant comments to complex code sections
  • I have updated documentation where needed
  • I have considered the security impact of these changes
  • I have considered performance implications
  • I have thought about error handling and edge cases
  • I have tested the changes in my local environment

Testing Instructions

Prerequisites

Steps to Test

Expected Results

Additional Context

Required Actions

  • Requires documentation updates
  • Requires downstream repository changes
  • Requires infrastructure/deployment changes
  • Requires coordination with other teams
  • Blocked by PR/MR: #XXX

Screenshots/Logs

Rohit and others added 11 commits November 14, 2025 20:28
Add connection manager architecture and test playbooks guide
658dba8 
- Add persistent connection manager architecture (ARCHITECTURE.md)
- Add implementation guide and developer documentation
- Add TEST_PLAYBOOKS.md with comprehensive test playbook examples
- Add base action plugin and platform manager implementation
- Add transformation framework and version management
- Add inventory and ansible.cfg for local testing
- Add PR guide for contributing workflow
@rohitthakur2590
Update manager_process
921f728 
Signed-off-by: rohitthakur2590 <rohitthakur2590@outlook.com>
@rohitthakur2590
update documentations
d43971e 
Signed-off-by: rohitthakur2590 <rohitthakur2590@outlook.com>
@rohitthakur2590
update arch diagrams
d13b29c 
Signed-off-by: rohitthakur2590 <rohitthakur2590@outlook.com>
@rohitthakur2590
add dataclasses docs
efa544a 
Signed-off-by: rohitthakur2590 <rohitthakur2590@outlook.com>
@rohitthakur2590
separate logic for fetching gateway
102c780 
Signed-off-by: rohitthakur2590 <rohitthakur2590@outlook.com>
@rohitthakur2590
update docs
2f1e3f0 
Signed-off-by: rohitthakur2590 <rohitthakur2590@outlook.com>
@rohitthakur2590
update docs
cce867f 
Signed-off-by: rohitthakur2590 <rohitthakur2590@outlook.com>
@rohitthakur2590
update docs
7620e2a 
Signed-off-by: rohitthakur2590 <rohitthakur2590@outlook.com>
@rohitthakur2590
add_perf_comp
225a362 
Signed-off-by: rohitthakur2590 <rohitthakur2590@outlook.com>
Implement_manager_process_tracking
1329197
@github-actions
Copy link

github-actions bot commented Dec 9, 2025

DVCS PR Check Results:

Could not find JIRA key(s) in PR title, branch name, or commit messages

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@AAYUSH2091 @rohitthakur2590

Comments