Skip to content

Add image digest glossary term - #3627

Merged
Okabe-Junya merged 8 commits into
cncf:mainfrom
VarshaUN:img-digest
Sep 22, 2026
Merged

Okabe-Junya merged 8 commits into
cncf:mainfrom
VarshaUN:img-digest

Conversation

@VarshaUN

Copy link
Copy Markdown
Contributor

Describe your changes

Related issue number or link (ex: resolves #issue-number)

This PR resolves kubernetes/website#55017

Checklist before opening this PR (put x in the checkboxes)

  • This PR does not contain plagiarism
    • don’t copy other people’s work unless you are quoting and contributing it to them.
  • I have signed off on all commits
    • signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

Signed-off-by : Varsha U N varshaun58@gmail.com

@netlify

netlify Bot commented Mar 26, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for cncfglossary ready!

Name Link
🔨 Latest commit b6234b7
🔍 Latest deploy log https://app.netlify.com/projects/cncfglossary/deploys/6aaa0c4406f06d00081dc668
😎 Deploy Preview https://deploy-preview-3627--cncfglossary.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@VarshaUN

VarshaUN commented Apr 3, 2026

Copy link
Copy Markdown
Contributor Author

@caniszczyk kindly review this

@Okabe-Junya Okabe-Junya left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you for your work 👍
left some comments, PTAL when you have time

Comment thread content/en/image-digest.md Outdated
Comment thread content/en/image-digest.md Outdated
Comment thread content/en/image-digest.md Outdated
Comment thread content/en/image-digest.md Outdated
Comment thread content/en/image-digest.md
Comment thread content/en/image-digest.md Outdated
Comment thread content/en/image-digest.md

@Okabe-Junya Okabe-Junya left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@lmktfy

lmktfy commented Jun 3, 2026

Copy link
Copy Markdown

@VarshaUN would you be willing to update this PR? Maybe do a rebase against main?

@VarshaUN

VarshaUN commented Aug 3, 2026

Copy link
Copy Markdown
Contributor Author

Hey @Okabe-Junya , I have adressed you review , kindly check it and lmk!

@Okabe-Junya Okabe-Junya left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Could you check CI error?
Looks some words are missing in wordlist (e.g.. SHA / cryptographic / reproducibility)

@Okabe-Junya Okabe-Junya left a comment •

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Sorry for inconvenience, could you check CI error again?
(would be better to in your local or create dummy PR in your fork)

@VarshaUN

Copy link
Copy Markdown
Contributor Author

Hey @Okabe-Junya , I have added the words correctly and CI has failed for unrelated spell check which is not related to my PR , can you help me with this?

@Okabe-Junya

Copy link
Copy Markdown
Collaborator

Ah, could you rebase?
Looks related to #3551 merge

@Okabe-Junya

Copy link
Copy Markdown
Collaborator

Ah, 3551 merged w/o CI run, so anyway need to put grpc in wordlist.txt

@Okabe-Junya

Copy link
Copy Markdown
Collaborator

So would be appreciated if you created another PR to add the word grpc in wordlist and then will rebase this PR

@VarshaUN

Copy link
Copy Markdown
Contributor Author

Hey @Okabe-Junya , checkout #3673

Signed-off-by: Varsha U N <varshaun58@gmail.com>
Signed-off-by: VarshaUN <varshaun58@gmail.com>
Signed-off-by: VarshaUN <varshaun58@gmail.com>
Signed-off-by: VarshaUN <varshaun58@gmail.com>
@VarshaUN

Copy link
Copy Markdown
Contributor Author

Hey @Okabe-Junya I have rebased and now checks have passed. Kindly review once!!

@nate-double-u nate-double-u left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for this @VarshaUN, could I ask for one more edit before we merged this in?

Comment thread content/en/image-digest.md Outdated
Updated the image digest documentation to clarify the concept and its importance in security and reproducibility.

Signed-off-by: Varsha U N <varshaun58@gmail.com>

@nate-double-u nate-double-u left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for the update @VarshaUN, I've integrated it a bit more into that first paragraph, and have done a copy edit pass throughout. Please have a look to make sure I haven't changed your meaning.

Comment thread content/en/image-digest.md Outdated
Comment thread content/en/image-digest.md Outdated
Comment thread content/en/image-digest.md Outdated
Comment thread content/en/image-digest.md Outdated
VarshaUN and others added 4 commits September 16, 2026 08:52
Co-authored-by: Nate W <natew@cncf.io>
Signed-off-by: Varsha U N <varshaun58@gmail.com>
Co-authored-by: Nate W <natew@cncf.io>
Signed-off-by: Varsha U N <varshaun58@gmail.com>
Co-authored-by: Nate W <natew@cncf.io>
Signed-off-by: Varsha U N <varshaun58@gmail.com>
Co-authored-by: Nate W <natew@cncf.io>
Signed-off-by: Varsha U N <varshaun58@gmail.com>

@nate-double-u nate-double-u left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks @VarshaUN!

Before we merge this, I'd like to ask @Okabe-Junya to re-review as this has changed a bit since he approved it.

@Okabe-Junya Okabe-Junya left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you!!

@Okabe-Junya
Okabe-Junya merged commit 5e5ac71 into cncf:main Sep 22, 2026
7 checks passed
@github-project-automation github-project-automation Bot moved this from 📋 Backlog to ✅ Done in CNCF glossary definition review board Sep 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lang/en for English

Projects

Development

Successfully merging this pull request may close these issues.

Link to Docker, about image digests not relevant

4 participants