Skip to content

Add RSA Key Automation System #309 - #461

Closed
7476ZHAO wants to merge 5 commits into
3C-SCSU:oldmainfrom
7476ZHAO:feature/rsa-key-auto-manager
Closed

Add RSA Key Automation System #309#461
7476ZHAO wants to merge 5 commits into
3C-SCSU:oldmainfrom
7476ZHAO:feature/rsa-key-auto-manager

Conversation

@7476ZHAO

Copy link
Copy Markdown
Contributor

Automated RSA key management system for Linux servers.
Adds cron-based SSH key expiry, cleanup, and metadata consistency handling (tested on GCP, deployed on standalone Linux).

@7476ZHAO 7476ZHAO changed the title Add RSA Key Automation System Add RSA Key Automation System #309 Oct 28, 2025

@3C-SCSU 3C-SCSU left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No code conflict.

@3C-SCSU 3C-SCSU left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are problems with the code submitted. Get in touch for more details.

@7476ZHAO 7476ZHAO closed this Oct 28, 2025
@7476ZHAO 7476ZHAO reopened this Oct 28, 2025
@7476ZHAO

Copy link
Copy Markdown
Contributor Author

There are problems with the code submitted. Get in touch for more details.

Dr. C, thank you for reminder. This problem occurs probably because GitHub Actions runners are hosted on Azure; when installing the filelock module, Ubuntu’s package manager automatically used the Azure mirror (azure.archive.ubuntu.com) for faster download, not because the code relies on Azure. I substituted filelock module with fcntl, tested again and commited again. Please try workflow one more. Thanks.

change the filelock part

@3C-SCSU 3C-SCSU left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code approval process -- 3 Steps review:
(1) Yes; (2) Maybe; (3) No; > {total lines changed - add/delete} [Reasoning]
example - feedback >> (2) > {341} [ Code needs to explain what it does ]

@3C-SCSU

3C-SCSU commented Nov 5, 2025

Copy link
Copy Markdown
Owner

Please ensure you've updated the Avatar code base with the latest changes from the rebased repository. After verifying consistency and confirming that your forked account and local clone are running the most current version, you can then resubmit the pull request.

@3C-SCSU 3C-SCSU closed this Nov 5, 2025
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.

2 participants