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

Fix for starting httpd on r3.4xlarge #108

Closed
wants to merge 1 commit into from
Closed

Fix for starting httpd on r3.4xlarge #108

wants to merge 1 commit into from

Conversation

jey
Copy link

@jey jey commented Mar 31, 2015

The merge of #76 improved things in terms of starting Apache on r3.4xlarge instances, but I also had to add this patch to fix the PHP module version before service httpd start worked successfully.

I have not tested this on other instance types. (Apparently there's some differences between the HVM and non-HVM AMIs?)

@shivaram
Copy link

@fredcons @nchammas -- Any thoughts on this ?

@nchammas
Copy link

Look innocuous, but I won't be able to test this.

Also, any reason this isn't being opened against branch-1.3 or branch-1.4?

v4 is for Spark 1.2.

@jey
Copy link
Author

jey commented Mar 31, 2015

I'm using Spark 1.2.1 and have only tested there

@fredcons
Copy link

fredcons commented Apr 1, 2015

Any idea on why the php version has changed ? A different underlying OS ?
In any case, the fix looks innocuous, as stated earlier by @nchammas

@jey
Copy link
Author

jey commented Apr 1, 2015

My hunch is that the HVM and non-HVM AMIs are being updated separately and got out of sync. (Just a guess -- I have not tried both.)

@shivaram
Copy link

shivaram commented Apr 2, 2015

Yeah the reason I am wary of merging this is that I don't want this to break non-HVM AMIs. @nchammas @fredcons does httpd work on HVM instances for you with any Spark version (1.2 / 1.3 etc.)

@shivaram
Copy link

@jey We have a newer PR for this in #121 which also installs the exact httpd version required -- so I'm closing this PR just to clear the queue for now.

@shivaram shivaram closed this Jul 21, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants