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

Provide means to reconfigure Apache/PHP/nginx on VMSS after initial deployment #121

Open
hosungsmsft opened this issue May 23, 2018 · 0 comments
Labels
enhancement New feature or request Priority 1 (may address) Pull Requests very welcome, but we are unlikely to get to this ourselves.

Comments

@hosungsmsft
Copy link

Currently it's not easy to reconfigure Apache/PHP/nginx on VMSS after initial deployment. #120 introduced a hook for a custom script (/moodle/bin/update-vmss-config) to run every minute. However, the author of such a script must be careful making the script idempotent (that is, executing the script multiple times should be the same as executing it just once, because it's run as a cron job), and make sure to delete the script after all the VMSS instances are updated. Deleting the script would be still problematic because any newly created instances will not be able to run the script. At this point, implementing idempotency in the script seems to be the only solution.

We should provide a better means to allow users to reconfigure Apache/PHP/nginx on VMSS after initial deployment. The html content update is done by /usr/local/bin/sync_moodle_html_local_copy_if_modified.sh that's triggered by update_last_modified_time.moodle_on_azure.sh on the controller machine. We should integrate these two into one and provide a uniform way of updating both the html content (php scripts) and the Apache/PHP/nginx configs as well. This is also related to #79, as we'll ultimately need some configuration file that'll control initial installation and later updates.

@SorraTheOrc SorraTheOrc added enhancement New feature or request Priority 1 (may address) Pull Requests very welcome, but we are unlikely to get to this ourselves. labels May 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Priority 1 (may address) Pull Requests very welcome, but we are unlikely to get to this ourselves.
Projects
None yet
Development

No branches or pull requests

2 participants