----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30133/#review69219 -----------------------------------------------------------
Ship it! Ship It! - Alejandro Fernandez On Jan. 22, 2015, 6:21 p.m., Yurii Shylov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/30133/ > ----------------------------------------------------------- > > (Updated Jan. 22, 2015, 6:21 p.m.) > > > Review request for Ambari, Alejandro Fernandez, Dmitro Lisnichenko, and Nate > Cole. > > > Bugs: AMBARI-9242 > https://issues.apache.org/jira/browse/AMBARI-9242 > > > Repository: ambari > > > Description > ------- > > API disallows Repo URLs from being edited if the version is current. > We allow this in 1.7.0 and is a regression. > We'll not be able to accommodate the case if the repo URL changes (say one > local mirror server to another, or from public URL to local mirror, etc). > > It is allowed always to change display_name. > It is possible to change upgrade_pack if it is not installed on any of the > clusters. > It is possible to always change base URLs. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/RepositoryVersionResourceProvider.java > a463439 > > ambari-server/src/test/java/org/apache/ambari/server/controller/internal/RepositoryVersionResourceProviderTest.java > 17fa2c2 > > Diff: https://reviews.apache.org/r/30133/diff/ > > > Testing > ------- > > Tests ok > > > Thanks, > > Yurii Shylov > >
