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

Use rsync instead of wget for mirroring group_vars and host_vars #34

Merged
merged 3 commits into from
Feb 1, 2017

Conversation

jabl
Copy link
Contributor

@jabl jabl commented Jan 30, 2017

With the help of the rsync --delete option, this gets rid of files
that have been deleted at the source, which can otherwise cause
problems.

Also, by using rsync instead of wget we save some of our precious
cluster internal bandwidth, since in the vast majority of cases there
are no changes.

jabl added 2 commits January 30, 2017 23:11
With the help of the rsync --delete option, this gets rid of files
that have been deleted at the source, which can otherwise cause
problems.

Also, by using rsync instead of wget we save some of our precious
cluster insternal bandwidth, since in the vast majority of cases there
are no changes.
@jabl
Copy link
Contributor Author

jabl commented Jan 31, 2017

We have this in use now, and it does work as advertised, although I haven't tried reinstalling a node yet. That being said, it should work, as git depends on rsync (repoquery --whatrequires rsync), and git is in the default kickstart packages list.

@jabl
Copy link
Contributor Author

jabl commented Jan 31, 2017

And yes, just tested that kickstarting also works.

@martbhell
Copy link
Contributor

Nice! If I can see clearly now there shouldn't be any issue á la #11 with this - nodes won't get ansible-pull-script.sh + rsync until an admin has run the fgci-install tag. And that tag/role also sets up the rsyncd.
According to manpage rsyncd "modules" are read-only so seems quite safe.

@jabl
Copy link
Contributor Author

jabl commented Jan 31, 2017

If one wants to be really paranoid one could move setting up rsyncd before generating the new ansible-pull-script.sh in service.yml..

And yes, the rsync daemon is read-only by default. There are ways to have some kind of user/password logins and write support, but it needs to be explicitly configured to be enabled.

@martbhell martbhell merged commit e4ff830 into fgci-org:master Feb 1, 2017
@martbhell martbhell removed the ready label Feb 1, 2017
martbhell added a commit that referenced this pull request Feb 1, 2017
@martbhell
Copy link
Contributor

In fgci-ansible/devel since fgci-org/fgci-ansible@a1700c0

As I didn't have a host_vars directory then it give an error. Nothing critical though:

Feb  1 09:15:03 io-install rsyncd[27614]: rsync: chroot /var/www/html/host_vars failed: No such file or directory (2)

Creating that directory in 154bf9f

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants