Koushik,
thanks for reporting this. You are probably using a database form before
this commit. This should work but I suspect it doesn't.
public <T> URI toUri(T value) {
try {
// do we need to check that value does not contain a scheme
// part?
try {
Long.parseLong(value.toString());
return new URI(Vlan.scheme + "://" + value);
} catch (NumberFormatException e) {
if (value.toString().contains(":"))
return new URI(value.toString());
else
return new URI(scheme, value.toString(), null);
}
} catch (URISyntaxException e) {
throw new CloudRuntimeException(
"Unable to convert to broadcast URI: " + value);
}
}
would be the correct code for the function that is your culprit, but master
doesn't work due to database mismatches between the code and the databse
schema. I will test this as soon as possible.
regards,
Daan
On Mon, Jul 22, 2013 at 5:18 PM, Koushik Das <[email protected]> wrote:
> Standard system VMs with systemvm.iso from latest master. But that
> shouldn't matter as the issue is in the deploy VM code.
>
> ________________________________
> From: Donal Lafferty
> Sent: 22/07/2013 6:37 PM
> To: '[email protected]'; [email protected]
> Subject: RE: System VMs not coming up due to
> https://reviews.apache.org/r/12685/
>
> Where are you getting your system VM from?
>
> DL
>
> > -----Original Message-----
> > From: Koushik Das [mailto:[email protected]]
> > Sent: 22 July 2013 11:09 AM
> > To: [email protected]
> > Subject: System VMs not coming up due to
> > https://reviews.apache.org/r/12685/
> >
> > Commit id: 2d4464d2badc9aff842fd180bafc4c384a83a91d
> >
> > - return new URI(scheme + "://" + value);
> > + // do we need to check that value does not contain a
> scheme
> > + // part?
> > + if (value.toString().contains(":"))
> > + return new URI(value.toString());
> > + else
> > + return new URI(scheme, value.toString(), null);
> >
> >
> > As part of this commit system VMs are not working in latest master. I
> see as
> > part of this change the broadcast/isolation URI format has changed from
> > vlan://<vlanid> to vlan:<vlanid>. The code in many places still depends
> on
> > the URI.getHosts() to extract the vlan id which is broken due to the new
> > format.
> > Can this be reverted?
> >
> >
> >
>
>
