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

Update faq.md #104

Closed
wants to merge 1 commit into from
Closed

Update faq.md #104

wants to merge 1 commit into from

Conversation

franciosi
Copy link

Just updating [backup] and [runtime reconfiguration] links

Just updating [backup] and [runtime reconfiguration] links
@@ -152,9 +152,9 @@ etcd sends a snapshot of its complete key-value store to refresh slow followers
[new_issue]: https://github.com/etcd-io/etcd/issues/new
[backend_commit_metrics]: ./metrics.md#disk
[raft]: https://raft.github.io/raft.pdf
[backup]: https://github.com/etcd-io/etcd/blob/master/Documentation/op-guide/recovery.md#snapshotting-the-keyspace
[backup]: https://github.com/etcd-io/website/blob/master/content/docs/v3.4.0/op-guide/recovery.md#snapshotting-the-keyspace
Copy link
Contributor

Choose a reason for hiding this comment

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

We can probably link directly to the page on site rather than linking to the repo here

Suggested change
[backup]: https://github.com/etcd-io/website/blob/master/content/docs/v3.4.0/op-guide/recovery.md#snapshotting-the-keyspace
[backup]: ./op-guide/recovery#snapshotting-the-keyspace

[chubby]: http://static.googleusercontent.com/media/research.google.com/en//archive/chubby-osdi06.pdf
[runtime reconfiguration]: https://github.com/etcd-io/etcd/blob/master/Documentation/op-guide/runtime-configuration.md
[runtime reconfiguration]: https://github.com/etcd-io/website/blob/master/content/docs/v3.4.0/op-guide/runtime-configuration.md
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
[runtime reconfiguration]: https://github.com/etcd-io/website/blob/master/content/docs/v3.4.0/op-guide/runtime-configuration.md
[runtime reconfiguration]: ./op-guide/runtime-configuration

@nate-double-u
Copy link
Contributor

Hi @franciosi, thanks for this. I have a couple suggestions in the file changed.

Would you also be willing to make these changes in the current directory as they will likely have the same issue?

@nate-double-u nate-double-u self-assigned this Mar 19, 2021
@franciosi
Copy link
Author

Yes, count on me

@nate-double-u
Copy link
Contributor

As there have been a lot of changes to the site, I'm closing this in favour of #203.

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