<div dir="ltr">Thanks Brian for the analysis and the fix.</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jun 28, 2017 at 10:14 PM, Brian Stansberry <span dir="ltr">&lt;<a href="mailto:brian.stansberry@redhat.com" target="_blank">brian.stansberry@redhat.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><a href="https://github.com/wildfly/wildfly/pull/10258" rel="noreferrer" target="_blank">https://github.com/wildfly/<wbr>wildfly/pull/10258</a> will result in the correct failure message when ‘delete’ is invoked against a non-existing resource.<br>
<div class="HOEnZb"><div class="h5"><br>
&gt; On Jun 28, 2017, at 12:36 PM, Brian Stansberry &lt;<a href="mailto:brian.stansberry@redhat.com">brian.stansberry@redhat.com</a>&gt; wrote:<br>
&gt;<br>
&gt; <a href="https://github.com/wildfly/wildfly/compare/master...bstansberry:tx-runtime-only" rel="noreferrer" target="_blank">https://github.com/wildfly/<wbr>wildfly/compare/master...<wbr>bstansberry:tx-runtime-only</a> is the quick improvement I mentioned.<br>
&gt;<br>
&gt; <a href="https://ci.wildfly.org/viewLog.html?buildId=66224&amp;tab=buildResultsDiv&amp;buildTypeId=WF_WildFlyCoreIntegrationExperiments" rel="noreferrer" target="_blank">https://ci.wildfly.org/<wbr>viewLog.html?buildId=66224&amp;<wbr>tab=buildResultsDiv&amp;<wbr>buildTypeId=WF_<wbr>WildFlyCoreIntegrationExperime<wbr>nts</a> is a custom CI job for that branch. (I’ve done no testing locally other than checking the transactions module compiles.)<br>
&gt;<br>
&gt; I don’t see how that op would get to ControllerLogger#<wbr>modelUpdateNotAuthorized though, unless the transactions=0\:ffff0afc4420\:<wbr>3b2568af\:594b9528\: resource doesn’t exist. If it doesn’t then the op would fail anyway, just with a different message.<br>
&gt;<br>
&gt;&gt; On Jun 28, 2017, at 12:19 PM, Michael Musgrove &lt;<a href="mailto:mmusgrov@redhat.com">mmusgrov@redhat.com</a>&gt; wrote:<br>
&gt;&gt;<br>
&gt;&gt; Thanks for looking into this. The operation they are running is:<br>
&gt;&gt;<br>
&gt;&gt; [<a href="http://domain@jboss7.pro.intra.rs.gov.br:9990" rel="noreferrer" target="_blank">domain@jboss7.pro.intra.rs.<wbr>gov.br:9990</a> /] /host=host-jbosshost7-1/<wbr>server=GPD1/subsystem=<wbr>transactions/log-store=log-<wbr>store/transactions=0\:<wbr>ffff0afc4420\:3b2568af\:<wbr>594b9528\::delete()<br>
&gt;&gt; {<br>
&gt;&gt;    &quot;outcome&quot; =&gt; &quot;failed&quot;,<br>
&gt;&gt;    &quot;result&quot; =&gt; undefined,<br>
&gt;&gt;    &quot;failure-description&quot; =&gt; &quot;WFLYCTL0249: Operação &#39;delete&#39; marcada no recurso &#39;[<br>
&gt;&gt;    (\&quot;subsystem\&quot; =&gt; \&quot;transactions\&quot;),<br>
&gt;&gt;    (\&quot;log-store\&quot; =&gt; \&quot;log-store\&quot;),<br>
&gt;&gt;    (\&quot;transactions\&quot; =&gt; \&quot;0:ffff0afc4420:3b2568af:<wbr>594b9528:\&quot;)<br>
&gt;&gt; ]&#39; foi diretamente invocada pelo usuário. As operações do usuário não são permitidas para atualizar diretamente a configuração persistente de um servidor no domain controller.&quot;,<br>
&gt;&gt;    &quot;rolled-back&quot; =&gt; true<br>
&gt;&gt; }<br>
&gt;&gt;<br>
&gt;&gt; On Wed, Jun 28, 2017 at 6:17 PM, Brian Stansberry &lt;<a href="mailto:brian.stansberry@redhat.com">brian.stansberry@redhat.com</a>&gt; wrote:<br>
&gt;&gt; What is the operation they are trying to invoke?<br>
&gt;&gt;<br>
&gt;&gt; LogStoreResource itself isn’t runtime-only but its children are. The “delete” operations on the “transactions” and “participants” children will trigger a check that could lead to ControllerLogger#<wbr>modelUpdateNotAuthorized, but those resources are runtime-only.<br>
&gt;&gt;<br>
&gt;&gt; One possibibility is the child they are trying to delete is already gone (these are dynamic) so perhaps since it’s non-existent the fact that it’s runtime-only is not being detected. I’ll work up a quick improvement that will help with that.<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt;&gt; On Jun 28, 2017, at 11:51 AM, Brian Stansberry &lt;<a href="mailto:brian.stansberry@redhat.com">brian.stansberry@redhat.com</a>&gt; wrote:<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; I’ll have a look.<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; On Jun 28, 2017, at 8:27 AM, Michael Musgrove &lt;<a href="mailto:mmusgrov@redhat.com">mmusgrov@redhat.com</a>&gt; wrote:<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; The operation used to work and I suspect that the root cause is WFCORE-2858 (Roll out READ_ONLY + RUNTIME_ONLY ops to the domain) and the fix for the transaction subsystem is <a href="https://issues.jboss.org/browse/JBEAP-11651" rel="noreferrer" target="_blank">https://issues.jboss.org/<wbr>browse/JBEAP-11651</a>. It would be great if someone with more knowledge than I of the domain model could verify my hypothesis.<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; On Wed, Jun 28, 2017 at 12:31 PM, Michael Musgrove &lt;<a href="mailto:mmusgrov@redhat.com">mmusgrov@redhat.com</a>&gt; wrote:<br>
&gt;&gt;&gt;&gt; We have a transaction case where the customer is trying to delete a (transaction log) model resource via the CLI and is seeing the error:<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; &quot;was directly invoked by the user. User operations are not permitted to directly update the persistent configuration of a server in a managed domain.&quot;<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; This is coming from ControllerLogger#<wbr>modelUpdateNotAuthorized (in wildfly-core) because we define the resource as *not* being runtime only:<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; public boolean isRuntime() {return false;}<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; My question is should we be defining the resource as a runtime resource or should we tell the customer not to delete the resouce from the controller?<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; Thanks,<br>
&gt;&gt;&gt;&gt; Mike<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; --<br>
&gt;&gt;&gt;&gt; Michael Musgrove<br>
&gt;&gt;&gt;&gt; Transactions Team<br>
&gt;&gt;&gt;&gt; e: <a href="mailto:mmusgrov@redhat.com">mmusgrov@redhat.com</a><br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; Our mission:To be the catalyst in communities of customers, contributors, and partners creating better technology the open source way.<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; JBoss, by Red Hat<br>
&gt;&gt;&gt;&gt; Registered Address: Red Hat Ltd, 6700 Cork Airport Business Park, Kinsale Road, Co. Cork.<br>
&gt;&gt;&gt;&gt; Registered in the Companies Registration Office, Parnell House, 14 Parnell Square, Dublin 1, Ireland, No.304873<br>
&gt;&gt;&gt;&gt; Directors:Michael Cunningham (USA), Vicky Wiseman (USA), Michael O&#39;Neill, Keith Phelan, Matt Parson (USA)<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; --<br>
&gt;&gt;&gt;&gt; Michael Musgrove<br>
&gt;&gt;&gt;&gt; Transactions Team<br>
&gt;&gt;&gt;&gt; e: <a href="mailto:mmusgrov@redhat.com">mmusgrov@redhat.com</a><br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; Our mission:To be the catalyst in communities of customers, contributors, and partners creating better technology the open source way.<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; JBoss, by Red Hat<br>
&gt;&gt;&gt;&gt; Registered Address: Red Hat Ltd, 6700 Cork Airport Business Park, Kinsale Road, Co. Cork.<br>
&gt;&gt;&gt;&gt; Registered in the Companies Registration Office, Parnell House, 14 Parnell Square, Dublin 1, Ireland, No.304873<br>
&gt;&gt;&gt;&gt; Directors:Michael Cunningham (USA), Vicky Wiseman (USA), Michael O&#39;Neill, Keith Phelan, Matt Parson (USA)<br>
&gt;&gt;&gt;&gt; ______________________________<wbr>_________________<br>
&gt;&gt;&gt;&gt; wildfly-dev mailing list<br>
&gt;&gt;&gt;&gt; <a href="mailto:wildfly-dev@lists.jboss.org">wildfly-dev@lists.jboss.org</a><br>
&gt;&gt;&gt;&gt; <a href="https://lists.jboss.org/mailman/listinfo/wildfly-dev" rel="noreferrer" target="_blank">https://lists.jboss.org/<wbr>mailman/listinfo/wildfly-dev</a><br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; --<br>
&gt;&gt;&gt; Brian Stansberry<br>
&gt;&gt;&gt; Manager, Senior Principal Software Engineer<br>
&gt;&gt;&gt; JBoss by Red Hat<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt; --<br>
&gt;&gt; Brian Stansberry<br>
&gt;&gt; Manager, Senior Principal Software Engineer<br>
&gt;&gt; JBoss by Red Hat<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt; --<br>
&gt;&gt; Michael Musgrove<br>
&gt;&gt; Transactions Team<br>
&gt;&gt; e: <a href="mailto:mmusgrov@redhat.com">mmusgrov@redhat.com</a><br>
&gt;&gt;<br>
&gt;&gt; Our mission:To be the catalyst in communities of customers, contributors, and partners creating better technology the open source way.<br>
&gt;&gt;<br>
&gt;&gt; JBoss, by Red Hat<br>
&gt;&gt; Registered Address: Red Hat Ltd, 6700 Cork Airport Business Park, Kinsale Road, Co. Cork.<br>
&gt;&gt; Registered in the Companies Registration Office, Parnell House, 14 Parnell Square, Dublin 1, Ireland, No.304873<br>
&gt;&gt; Directors:Michael Cunningham (USA), Vicky Wiseman (USA), Michael O&#39;Neill, Keith Phelan, Matt Parson (USA)<br>
&gt;<br>
&gt; --<br>
&gt; Brian Stansberry<br>
&gt; Manager, Senior Principal Software Engineer<br>
&gt; JBoss by Red Hat<br>
&gt;<br>
&gt;<br>
&gt;<br>
<br>
--<br>
Brian Stansberry<br>
Manager, Senior Principal Software Engineer<br>
JBoss by Red Hat<br>
<br>
<br>
<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><div>Michael Musgrove</div><div>Transactions Team</div><div>e: <a href="mailto:mmusgrov@redhat.com" target="_blank">mmusgrov@redhat.com</a></div><div><br></div><div>Our mission:To be the catalyst in communities of customers, contributors, and partners creating better technology the open source way. <br></div><div><br></div><div><div>JBoss, by Red Hat</div><div>Registered Address: Red Hat Ltd, 6700 Cork Airport Business Park, Kinsale Road, Co. Cork.</div><div>Registered in the Companies Registration Office, Parnell House, 14 Parnell Square, Dublin 1, Ireland, No.304873</div><div>Directors:Michael Cunningham (USA), Vicky Wiseman (USA), Michael O&#39;Neill, Keith Phelan, Matt Parson (USA)</div></div></div></div></div></div></div>
</div>