-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25023/#review51588
-----------------------------------------------------------
./systemvm/patches/debian/config/var/www/html/latest/.htaccess
That file has a stub-version of the file, and is pre-seeded with one rewrite
rule...
looks like this:
Options +FollowSymLinks
RewriteEngine On
#RewriteBase /
RewriteRule ^user-data$ ../userdata/%{REMOTE_ADDR}/user-data [L,NC,QSA]
That rule also probably needs to be updated.
You might also want to look at
https://reviews.apache.org/r/25065/
and perhaps we could combine our patches...
- Fred Clift
On Aug. 25, 2014, 7:55 p.m., Erik Weber wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25023/
> -----------------------------------------------------------
>
> (Updated Aug. 25, 2014, 7:55 p.m.)
>
>
> Review request for cloudstack, Marcus Sorensen, Sebastien Goasguen, and Wido
> den Hollander.
>
>
> Bugs: CLOUDSTACK-7405
> https://issues.apache.org/jira/browse/CLOUDSTACK-7405
>
>
> Repository: cloudstack-git
>
>
> Description
> -------
>
> As per https://issues.apache.org/jira/browse/CLOUDSTACK-7405 cloud-init
> expects to be able to get meta-data directory without using a trailing slash.
>
> Ultimately this should be fixed in cloud-init, but it's an unintrusive fix in
> cloudstack
>
>
> Diffs
> -----
>
> systemvm/patches/debian/config/opt/cloud/bin/vmdata.py f508032
>
> Diff: https://reviews.apache.org/r/25023/diff/
>
>
> Testing
> -------
>
> tested with curl that both new and old url works
>
> [root@jenkins ~]# curl -I -s 10.30.81.1/latest/meta-data/vm-id | grep HTTP
> HTTP/1.1 200 OK
> [root@jenkins ~]# curl -I -s 10.30.81.1/latest/meta-data | grep HTTP
> HTTP/1.1 200 OK
>
>
> Thanks,
>
> Erik Weber
>
>