-
Notifications
You must be signed in to change notification settings - Fork 100
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
Rename API redirects httpd conf file #221
Rename API redirects httpd conf file #221
Conversation
This goes along with the changes in ManageIQ/manageiq-appliance#131
@@ -12,7 +12,7 @@ Options SymLinksIfOwnerMatch | |||
<VirtualHost *:80> | |||
DocumentRoot /var/www/miq/vmdb/public | |||
Include conf.d/manageiq-redirects-ui | |||
Include conf.d/manageiq-redirects-ws | |||
Include conf.d/manageiq-redirects-api |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In the other repo, I think there was an ordering issue that forced me to put api before ui, so that should be done here as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I imagine it wouldn't matter. This is working now and you didn't change the ordering in the PR where you made the name change right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I ensured the ordering was api, then ui, then websocket because that's what it was. I remember @bdunne saying order was important (though I don't know why).
Checked commits carbonin/manageiq-pods@c8c936f~...920f6f4 with ruby 2.3.3, rubocop 0.47.1, and haml-lint 0.20.0 |
🎉 Thanks @bdunne |
This goes along with the changes in ManageIQ/manageiq-appliance#131
This should be merged after that patch so that the file actually exists.