You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by Dmitro Lisnichenko <dl...@hortonworks.com> on 2015/09/21 18:40:55 UTC
Review Request 38559: [Upgrade] Check CONFIG_MERGE failed error
message is consistently reported in ambari-server log
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38559/
-----------------------------------------------------------
Review request for Ambari, Alejandro Fernandez, Andrew Onischuk, Jaimin Jetly, Jonathan Hurley, and Nate Cole.
Bugs: AMBARI-13152
https://issues.apache.org/jira/browse/AMBARI-13152
Repository: ambari
Description
-------
Current issue is caused by commit AMBARI-13107 ("After successful install of Oozie via blueprint, oozie has an invalid configuration") . Mentioned commit removes few "value" sections from service metainfo. That's why some properties in stack metainfo may have null values.
For now, we are going to revert an offending patch from 2.1.2, so probably current patch if going to go only to branch-2.1 or to trunk
Diffs
-----
ambari-server/src/main/java/org/apache/ambari/server/state/ConfigMergeHelper.java 9a2c604
Diff: https://reviews.apache.org/r/38559/diff/
Testing
-------
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] Ambari Views ...................................... SUCCESS [3.262s]
[INFO] Ambari Metrics Common ............................. SUCCESS [1.659s]
[INFO] Ambari Server ..................................... SUCCESS [53:43.587s]
[INFO] Ambari Agent ...................................... SUCCESS [9.281s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 53:58.376s
[INFO] Finished at: Mon Sep 21 18:54:27 EEST 2015
[INFO] Final Memory: 73M/1692M
Thanks,
Dmitro Lisnichenko
Re: Review Request 38559: [Upgrade] Check CONFIG_MERGE failed error
message is consistently reported in ambari-server log
Posted by Nate Cole <nc...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38559/#review99774
-----------------------------------------------------------
Ship it!
Ship It!
- Nate Cole
On Sept. 21, 2015, 1:19 p.m., Dmitro Lisnichenko wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/38559/
> -----------------------------------------------------------
>
> (Updated Sept. 21, 2015, 1:19 p.m.)
>
>
> Review request for Ambari, Alejandro Fernandez, Andrew Onischuk, Jaimin Jetly, Jonathan Hurley, and Nate Cole.
>
>
> Bugs: AMBARI-13152
> https://issues.apache.org/jira/browse/AMBARI-13152
>
>
> Repository: ambari
>
>
> Description
> -------
>
> Current issue is caused by commit AMBARI-13107 ("After successful install of Oozie via blueprint, oozie has an invalid configuration") . Mentioned commit removes few "value" sections from service metainfo. That's why some properties in stack metainfo may have null values.
>
> A litle more details about the case I'm fixing:
> Both previous stack value and next stack value are null (not defined in metainfo to avoid parsing by blueprint). But runtime property value after upgrade is not-null (added by UpgradeCatalog). I'm fixing exactly this case, other cases (only new or old stack property value is null and other is not) are a way different because of bug nature.
>
> UPD we decided to commit the fix to 2.1.2 as well since mentioned patch is hard to revert (other patches are based on it)
>
>
> Diffs
> -----
>
> ambari-server/src/main/java/org/apache/ambari/server/state/ConfigMergeHelper.java 9a2c604
>
> Diff: https://reviews.apache.org/r/38559/diff/
>
>
> Testing
> -------
>
> [INFO] ------------------------------------------------------------------------
> [INFO] Reactor Summary:
> [INFO]
> [INFO] Ambari Views ...................................... SUCCESS [3.262s]
> [INFO] Ambari Metrics Common ............................. SUCCESS [1.659s]
> [INFO] Ambari Server ..................................... SUCCESS [53:43.587s]
> [INFO] Ambari Agent ...................................... SUCCESS [9.281s]
> [INFO] ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] ------------------------------------------------------------------------
> [INFO] Total time: 53:58.376s
> [INFO] Finished at: Mon Sep 21 18:54:27 EEST 2015
> [INFO] Final Memory: 73M/1692M
>
>
> Thanks,
>
> Dmitro Lisnichenko
>
>
Re: Review Request 38559: [Upgrade] Check CONFIG_MERGE failed error
message is consistently reported in ambari-server log
Posted by Mahadev Konar <ma...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38559/#review99769
-----------------------------------------------------------
Ship it!
Ship It!
- Mahadev Konar
On Sept. 21, 2015, 5:19 p.m., Dmitro Lisnichenko wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/38559/
> -----------------------------------------------------------
>
> (Updated Sept. 21, 2015, 5:19 p.m.)
>
>
> Review request for Ambari, Alejandro Fernandez, Andrew Onischuk, Jaimin Jetly, Jonathan Hurley, and Nate Cole.
>
>
> Bugs: AMBARI-13152
> https://issues.apache.org/jira/browse/AMBARI-13152
>
>
> Repository: ambari
>
>
> Description
> -------
>
> Current issue is caused by commit AMBARI-13107 ("After successful install of Oozie via blueprint, oozie has an invalid configuration") . Mentioned commit removes few "value" sections from service metainfo. That's why some properties in stack metainfo may have null values.
>
> A litle more details about the case I'm fixing:
> Both previous stack value and next stack value are null (not defined in metainfo to avoid parsing by blueprint). But runtime property value after upgrade is not-null (added by UpgradeCatalog). I'm fixing exactly this case, other cases (only new or old stack property value is null and other is not) are a way different because of bug nature.
>
> UPD we decided to commit the fix to 2.1.2 as well since mentioned patch is hard to revert (other patches are based on it)
>
>
> Diffs
> -----
>
> ambari-server/src/main/java/org/apache/ambari/server/state/ConfigMergeHelper.java 9a2c604
>
> Diff: https://reviews.apache.org/r/38559/diff/
>
>
> Testing
> -------
>
> [INFO] ------------------------------------------------------------------------
> [INFO] Reactor Summary:
> [INFO]
> [INFO] Ambari Views ...................................... SUCCESS [3.262s]
> [INFO] Ambari Metrics Common ............................. SUCCESS [1.659s]
> [INFO] Ambari Server ..................................... SUCCESS [53:43.587s]
> [INFO] Ambari Agent ...................................... SUCCESS [9.281s]
> [INFO] ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] ------------------------------------------------------------------------
> [INFO] Total time: 53:58.376s
> [INFO] Finished at: Mon Sep 21 18:54:27 EEST 2015
> [INFO] Final Memory: 73M/1692M
>
>
> Thanks,
>
> Dmitro Lisnichenko
>
>
Re: Review Request 38559: [Upgrade] Check CONFIG_MERGE failed error
message is consistently reported in ambari-server log
Posted by Alejandro Fernandez <af...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38559/#review99766
-----------------------------------------------------------
Ship it!
Ship It!
- Alejandro Fernandez
On Sept. 21, 2015, 5:19 p.m., Dmitro Lisnichenko wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/38559/
> -----------------------------------------------------------
>
> (Updated Sept. 21, 2015, 5:19 p.m.)
>
>
> Review request for Ambari, Alejandro Fernandez, Andrew Onischuk, Jaimin Jetly, Jonathan Hurley, and Nate Cole.
>
>
> Bugs: AMBARI-13152
> https://issues.apache.org/jira/browse/AMBARI-13152
>
>
> Repository: ambari
>
>
> Description
> -------
>
> Current issue is caused by commit AMBARI-13107 ("After successful install of Oozie via blueprint, oozie has an invalid configuration") . Mentioned commit removes few "value" sections from service metainfo. That's why some properties in stack metainfo may have null values.
>
> A litle more details about the case I'm fixing:
> Both previous stack value and next stack value are null (not defined in metainfo to avoid parsing by blueprint). But runtime property value after upgrade is not-null (added by UpgradeCatalog). I'm fixing exactly this case, other cases (only new or old stack property value is null and other is not) are a way different because of bug nature.
>
> UPD we decided to commit the fix to 2.1.2 as well since mentioned patch is hard to revert (other patches are based on it)
>
>
> Diffs
> -----
>
> ambari-server/src/main/java/org/apache/ambari/server/state/ConfigMergeHelper.java 9a2c604
>
> Diff: https://reviews.apache.org/r/38559/diff/
>
>
> Testing
> -------
>
> [INFO] ------------------------------------------------------------------------
> [INFO] Reactor Summary:
> [INFO]
> [INFO] Ambari Views ...................................... SUCCESS [3.262s]
> [INFO] Ambari Metrics Common ............................. SUCCESS [1.659s]
> [INFO] Ambari Server ..................................... SUCCESS [53:43.587s]
> [INFO] Ambari Agent ...................................... SUCCESS [9.281s]
> [INFO] ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] ------------------------------------------------------------------------
> [INFO] Total time: 53:58.376s
> [INFO] Finished at: Mon Sep 21 18:54:27 EEST 2015
> [INFO] Final Memory: 73M/1692M
>
>
> Thanks,
>
> Dmitro Lisnichenko
>
>
Re: Review Request 38559: [Upgrade] Check CONFIG_MERGE failed error
message is consistently reported in ambari-server log
Posted by Dmitro Lisnichenko <dl...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38559/
-----------------------------------------------------------
(Updated Sept. 21, 2015, 5:19 p.m.)
Review request for Ambari, Alejandro Fernandez, Andrew Onischuk, Jaimin Jetly, Jonathan Hurley, and Nate Cole.
Bugs: AMBARI-13152
https://issues.apache.org/jira/browse/AMBARI-13152
Repository: ambari
Description (updated)
-------
Current issue is caused by commit AMBARI-13107 ("After successful install of Oozie via blueprint, oozie has an invalid configuration") . Mentioned commit removes few "value" sections from service metainfo. That's why some properties in stack metainfo may have null values.
A litle more details about the case I'm fixing:
Both previous stack value and next stack value are null (not defined in metainfo to avoid parsing by blueprint). But runtime property value after upgrade is not-null (added by UpgradeCatalog). I'm fixing exactly this case, other cases (only new or old stack property value is null and other is not) are a way different because of bug nature.
UPD we decided to commit the fix to 2.1.2 as well since mentioned patch is hard to revert (other patches are based on it)
Diffs
-----
ambari-server/src/main/java/org/apache/ambari/server/state/ConfigMergeHelper.java 9a2c604
Diff: https://reviews.apache.org/r/38559/diff/
Testing
-------
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] Ambari Views ...................................... SUCCESS [3.262s]
[INFO] Ambari Metrics Common ............................. SUCCESS [1.659s]
[INFO] Ambari Server ..................................... SUCCESS [53:43.587s]
[INFO] Ambari Agent ...................................... SUCCESS [9.281s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 53:58.376s
[INFO] Finished at: Mon Sep 21 18:54:27 EEST 2015
[INFO] Final Memory: 73M/1692M
Thanks,
Dmitro Lisnichenko
Re: Review Request 38559: [Upgrade] Check CONFIG_MERGE failed error
message is consistently reported in ambari-server log
Posted by Dmitro Lisnichenko <dl...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38559/
-----------------------------------------------------------
(Updated Sept. 21, 2015, 5:11 p.m.)
Review request for Ambari, Alejandro Fernandez, Andrew Onischuk, Jaimin Jetly, Jonathan Hurley, and Nate Cole.
Bugs: AMBARI-13152
https://issues.apache.org/jira/browse/AMBARI-13152
Repository: ambari
Description (updated)
-------
Current issue is caused by commit AMBARI-13107 ("After successful install of Oozie via blueprint, oozie has an invalid configuration") . Mentioned commit removes few "value" sections from service metainfo. That's why some properties in stack metainfo may have null values.
UPD we decided to commit the fix to 2.1.2 as well since mentioned patch is hard to revert (other patches are based on it)
Diffs
-----
ambari-server/src/main/java/org/apache/ambari/server/state/ConfigMergeHelper.java 9a2c604
Diff: https://reviews.apache.org/r/38559/diff/
Testing
-------
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] Ambari Views ...................................... SUCCESS [3.262s]
[INFO] Ambari Metrics Common ............................. SUCCESS [1.659s]
[INFO] Ambari Server ..................................... SUCCESS [53:43.587s]
[INFO] Ambari Agent ...................................... SUCCESS [9.281s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 53:58.376s
[INFO] Finished at: Mon Sep 21 18:54:27 EEST 2015
[INFO] Final Memory: 73M/1692M
Thanks,
Dmitro Lisnichenko
Re: Review Request 38559: [Upgrade] Check CONFIG_MERGE failed error
message is consistently reported in ambari-server log
Posted by Dmitro Lisnichenko <dl...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38559/
-----------------------------------------------------------
(Updated Sept. 21, 2015, 4:41 p.m.)
Review request for Ambari, Alejandro Fernandez, Andrew Onischuk, Jaimin Jetly, Jonathan Hurley, and Nate Cole.
Bugs: AMBARI-13152
https://issues.apache.org/jira/browse/AMBARI-13152
Repository: ambari
Description (updated)
-------
Current issue is caused by commit AMBARI-13107 ("After successful install of Oozie via blueprint, oozie has an invalid configuration") . Mentioned commit removes few "value" sections from service metainfo. That's why some properties in stack metainfo may have null values.
For now, we are going to revert an offending patch from 2.1.2, so probably current patch if going to go only to branch-2.1 and to trunk
Diffs
-----
ambari-server/src/main/java/org/apache/ambari/server/state/ConfigMergeHelper.java 9a2c604
Diff: https://reviews.apache.org/r/38559/diff/
Testing
-------
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] Ambari Views ...................................... SUCCESS [3.262s]
[INFO] Ambari Metrics Common ............................. SUCCESS [1.659s]
[INFO] Ambari Server ..................................... SUCCESS [53:43.587s]
[INFO] Ambari Agent ...................................... SUCCESS [9.281s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 53:58.376s
[INFO] Finished at: Mon Sep 21 18:54:27 EEST 2015
[INFO] Final Memory: 73M/1692M
Thanks,
Dmitro Lisnichenko