Skip to content
This repository has been archived by the owner on Jan 30, 2020. It is now read-only.

docs: remove D-Bus and polkit note, it is not true. #1427

Merged
merged 1 commit into from
Feb 29, 2016

Conversation

kayrus
Copy link
Contributor

@kayrus kayrus commented Feb 5, 2016

/cc @jonboulle
This wiki page is outdated http://www.freedesktop.org/wiki/Software/systemd/dbus/
Polkit rule was already implemented in this PR: coreos/coreos-overlay#1579

@jonboulle
Copy link
Contributor

oh really? interesting. but can we change this to a note about how packagers/deployers can do this then?

@jonboulle
Copy link
Contributor

And worth pinging upstream systemd about that doc, I don't know where they keep the source..

@kayrus
Copy link
Contributor Author

kayrus commented Feb 9, 2016

systemd/systemd#2562

@kayrus
Copy link
Contributor Author

kayrus commented Feb 19, 2016

systemd/systemd#1042

@jonboulle
Copy link
Contributor

perhaps #1427 (comment) and then we can move forward with this?

@kayrus kayrus added this to the v0.12.0 milestone Feb 29, 2016
@kayrus kayrus self-assigned this Feb 29, 2016
@kayrus kayrus force-pushed the kayrus/remove_dbus_polkit_note branch from c29e546 to ab8abcb Compare February 29, 2016 10:10
@kayrus
Copy link
Contributor Author

kayrus commented Feb 29, 2016

@jonboulle updated doc. added more security notes.

@kayrus
Copy link
Contributor Author

kayrus commented Feb 29, 2016

I've also added new fleetd option into docs.

[etcd]: https://coreos.com/docs/cluster-management/setup/getting-started-with-etcd
### TLS Authentication

If you have etcd cluster with enabled TLS authentication you have to configure fleet to use TLS keypair. This example will should you how to do that:
Copy link
Contributor

Choose a reason for hiding this comment

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

If your etcd cluster has TLS authentication enabled, you will need to configure fleet to use an appropriate TLS keypair. The example shows below how to achieve this:

@jonboulle
Copy link
Contributor

A few minor tweaks and needs a rebase. LGTM after that.

@kayrus kayrus force-pushed the kayrus/remove_dbus_polkit_note branch 3 times, most recently from bbcb4b0 to fb88d7d Compare February 29, 2016 11:40
@kayrus kayrus force-pushed the kayrus/remove_dbus_polkit_note branch from fb88d7d to aae37e1 Compare February 29, 2016 11:55
@kayrus
Copy link
Contributor Author

kayrus commented Feb 29, 2016

Done. @jonboulle final review?

@jonboulle
Copy link
Contributor

lgtm

jonboulle added a commit that referenced this pull request Feb 29, 2016
docs: remove D-Bus and polkit note, it is not true.
@jonboulle jonboulle merged commit 1550074 into coreos:master Feb 29, 2016
@jonboulle
Copy link
Contributor

thanks

@tixxdz tixxdz mentioned this pull request Mar 1, 2016
@kayrus kayrus deleted the kayrus/remove_dbus_polkit_note branch March 30, 2016 07:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants