----------------------------------------------------------- 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 > >