Skip to content

drop the usage of the E2E_SSH_USER secret#1889

Merged
tariq1890 merged 1 commit into
mainfrom
rm-e2e-ssh-user-secret
Jun 16, 2026
Merged

drop the usage of the E2E_SSH_USER secret#1889
tariq1890 merged 1 commit into
mainfrom
rm-e2e-ssh-user-secret

Conversation

@tariq1890

Copy link
Copy Markdown
Contributor

"ubuntu" being the SSH user of AWS Ubuntu EC2 instances is a well-known fact. There is not much to gain from persisting this value as a secret.

Source: https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/managing-users.html

@tariq1890 tariq1890 requested a review from cdesiniotis June 15, 2026 19:39
@coveralls

coveralls commented Jun 15, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 27589237511

Coverage remained the same at 43.541%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 15009
Covered Lines: 6535
Line Coverage: 43.54%
Coverage Strength: 0.48 hits per line

💛 - Coveralls

@tariq1890 tariq1890 enabled auto-merge June 16, 2026 01:16
"ubuntu" being the SSH user of AWS Ubuntu EC2 instances is a
well-known fact. There is not much to gain from persisting
this value as a secret.

Source: https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/managing-users.html

Signed-off-by: Tariq Ibrahim <tibrahim@nvidia.com>
@tariq1890 tariq1890 force-pushed the rm-e2e-ssh-user-secret branch from a51fa7a to 4d77ce7 Compare June 16, 2026 02:06
@tariq1890 tariq1890 merged commit af0aaa4 into main Jun 16, 2026
20 checks passed
@tariq1890 tariq1890 deleted the rm-e2e-ssh-user-secret branch June 16, 2026 02:36
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