Change SSH key to SHA256 format

[Nihalsaeed]

Updated SSH key format in containers.yml.

Summary by CodeRabbit

• Chores
  • Updated SSH authentication configuration for a container to replace an existing public key with a new public key value.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 4538d166-d277-42ae-818c-a1fc5cc62229

📥 Commits

Reviewing files that changed from the base of the PR and between efba86d and 1f64d4b.

📒 Files selected for processing (1)

• containers/containers.yml

✅ Files skipped from review due to trivial changes (1)

• containers/containers.yml

---

📝 Walkthrough

Walkthrough

The containers/containers.yml entry for member-nihal updates the ssh_key value by replacing a placeholder ssh-ed25519 AAAA... string with a concrete ssh-ed25519 public key that includes the key material and comment tharilla.

Changes

Cohort / File(s)	Summary
Container Configuration containers/containers.yml	Replaced placeholder ssh_key for member-nihal with a specific ssh-ed25519 public key string (includes full key material and comment tharilla).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐇 I hopped in the config, made a small tweak,
Swapped a shy placeholder for a key unique,
ssh-ed25519 now sings with tharilla bright,
Containers connect smoothly through the night,
A tiny change, and everything's alright.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	The title claims the change converts SSH key to SHA256 format, but the actual change only replaces one ssh-ed25519 key with another ssh-ed25519 key without any format conversion to SHA256.	Update the title to accurately reflect the change, such as 'Update SSH key for member-nihal container' or 'Replace SSH key in containers.yml'.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

⚔️ Resolve merge conflicts

• Resolve merge conflict in branch main

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@containers/containers.yml`:
- Line 9: Replace the SSH fingerprint string in the ssh_key value with a full
OpenSSH public key line because scripts/container.sh writes ssh_key directly
into authorized_keys (see ssh_key and the authorized_keys write in
scripts/container.sh around line 39); update the value from the SHA256
fingerprint to a complete key (e.g., starting with "ssh-ed25519" or "ssh-rsa"
followed by the base64 blob and optional comment) so OpenSSH will accept it when
the container initialization writes to authorized_keys.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: e471474f-7f3d-4146-9771-bcc22219d89b

📥 Commits

Reviewing files that changed from the base of the PR and between 67ef1bc and efba86d.

📒 Files selected for processing (1)

• containers/containers.yml