search

fedora-infra / mirrormanager2

Rewrite of the MirrorManager application in Flask and SQLAlchemy
https://mirrormanager.fedoraproject.org
GNU General Public License v2.0
63 stars 46 forks source link

Host templates: Remove incorrect statements #163

Closed srisankethu closed 8 years ago

srisankethu commented 8 years ago

Fixes https://github.com/fedora-infra/mirrormanager2/issues/145

pypingou commented 8 years ago

@adrianreber, looks good to me, and to you?

adrianreber commented 8 years ago

I probably did not describe my intentions in the issue good enough.

The goal was to remove the complete "Master rsync server Access Control List IPs" section.

I would also make it configurable if the MirrorManager installation has this feature or not. So it should be configurable. Other projects (which might be interested in using MirrorManager) might use this functionality. For Fedora's setup, with its tiering, it just does not make much sense.

srisankethu commented 8 years ago

@adrianreber I am a beginner...Can you tell me what changes can i make in these files?

ralphbean commented 8 years ago

Hm, I'm going to close this one since it seems like it doesn't match up with @adrianreber's ideas in #163.

@srisankethu thanks for submitting it! Can you follow up in #163 and ask for more details there about what should be fixed?

ralphbean commented 8 years ago

Sorry.. I meant #145, there.

srisankethu commented 8 years ago

@adrianreber :+1: