DK101010 commented on issue #5202: URL: https://github.com/apache/cloudstack/issues/5202#issuecomment-881325896
> @DK101010 > > > Hi @nvazquez, this issue is needed for my PR in that case of existing community networks lower 4.15.0.0. But it could be necessary to add this as soon as possible to cloudstack because all already existing community networks in cloudstack are wrong. I can not estimate which impact it has for other CS functionality beside the ingest. > > are you saying that that PR won't work without a DB upgrade for all URI fields? (bit worried here) Hi Daan, sure. The current implementation always expects an "i" in the broadcast uri and throws errors for community networks that are created in CS 4.15.0.0 or above. My PR add an "i" or a "c" depending on the pvlan type and would throws errors for community networks that are created in 4.14. and below. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
