Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated instructions for acessing (reading and writing) an etcd KV pair #812

Merged
merged 1 commit into from Mar 28, 2024

Conversation

dwelsch-esi
Copy link
Contributor

@dwelsch-esi dwelsch-esi commented Mar 19, 2024

Resolves #790.

Copy link
Member

@jmhbnz jmhbnz left a comment

Choose a reason for hiding this comment

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

Hey @dwelsch-esi - Thanks for raising this update. To enable us to merge this can you please ensure your commit is signed so our developer certificate of origin check passes.

For example:

git rebase HEAD~2 --signoff
git push --force

Additionally - I would suggest we update the v3.5 version of these tutorials to match.

@dwelsch-esi
Copy link
Contributor Author

DCO corrected.

@jmhbnz
Copy link
Member

jmhbnz commented Mar 26, 2024

DCO corrected.

Hi @dwelsch-esi - Reviewing the diff it looks like there are now unintended changes showing? Additionally please squash commits to ensure only one commit is present.

- Break `how-to-access-etcd` into two files:
  - `writing-to-etcd`
  - `reading-from-etcd`
- Rename pages from "How to access ..." to "Writing ..." and "Reading ..."
- Added Prerequisite section
- Removed animated GIF
- Revised procedure to explain keywords

Create reading-from-etcd.md

updating v3.5 with same v3.6 content as above.

Signed-off-by: Nate W <natew@cncf.io>
@nate-double-u
Copy link
Contributor

@jmhbnz I've helped Dave to rebase this and update for both v3.5 and v3.6.

Copy link
Member

@jmhbnz jmhbnz left a comment

Choose a reason for hiding this comment

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

LGTM - Thanks team

@nate-double-u nate-double-u merged commit dd230ad into etcd-io:main Mar 28, 2024
5 checks passed
ivanvc pushed a commit to ivanvc/etcd-website that referenced this pull request May 13, 2024
- Break `how-to-access-etcd` into two files:
  - `writing-to-etcd`
  - `reading-from-etcd`
- Rename pages from "How to access ..." to "Writing ..." and "Reading ..."
- Added Prerequisite section
- Removed animated GIF
- Revised procedure to explain keywords

Create reading-from-etcd.md

updating v3.5 with same v3.6 content as above.

Signed-off-by: Nate W <natew@cncf.io>
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.

Revise https://etcd.io/docs/v3.5/tutorials/how-to-access-etcd/
3 participants