<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"><<a href="mailto:brian.stansberry@redhat.com" target="_blank">brian.stansberry@redhat.com</a>></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>
> On Jun 28, 2017, at 12:36 PM, Brian Stansberry <<a href="mailto:brian.stansberry@redhat.com">brian.stansberry@redhat.com</a>> wrote:<br>
><br>
> <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>
><br>
> <a href="https://ci.wildfly.org/viewLog.html?buildId=66224&tab=buildResultsDiv&buildTypeId=WF_WildFlyCoreIntegrationExperiments" rel="noreferrer" target="_blank">https://ci.wildfly.org/<wbr>viewLog.html?buildId=66224&<wbr>tab=buildResultsDiv&<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>
><br>
> 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>
><br>
>> On Jun 28, 2017, at 12:19 PM, Michael Musgrove <<a href="mailto:mmusgrov@redhat.com">mmusgrov@redhat.com</a>> wrote:<br>
>><br>
>> Thanks for looking into this. The operation they are running is:<br>
>><br>
>> [<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>
>> {<br>
>> "outcome" => "failed",<br>
>> "result" => undefined,<br>
>> "failure-description" => "WFLYCTL0249: Operação 'delete' marcada no recurso '[<br>
>> (\"subsystem\" => \"transactions\"),<br>
>> (\"log-store\" => \"log-store\"),<br>
>> (\"transactions\" => \"0:ffff0afc4420:3b2568af:<wbr>594b9528:\")<br>
>> ]' 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.",<br>
>> "rolled-back" => true<br>
>> }<br>
>><br>
>> On Wed, Jun 28, 2017 at 6:17 PM, Brian Stansberry <<a href="mailto:brian.stansberry@redhat.com">brian.stansberry@redhat.com</a>> wrote:<br>
>> What is the operation they are trying to invoke?<br>
>><br>
>> 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>
>><br>
>> 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>
>><br>
>><br>
>>> On Jun 28, 2017, at 11:51 AM, Brian Stansberry <<a href="mailto:brian.stansberry@redhat.com">brian.stansberry@redhat.com</a>> wrote:<br>
>>><br>
>>> I’ll have a look.<br>
>>><br>
>>>> On Jun 28, 2017, at 8:27 AM, Michael Musgrove <<a href="mailto:mmusgrov@redhat.com">mmusgrov@redhat.com</a>> wrote:<br>
>>>><br>
>>>> 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>
>>>><br>
>>>> On Wed, Jun 28, 2017 at 12:31 PM, Michael Musgrove <<a href="mailto:mmusgrov@redhat.com">mmusgrov@redhat.com</a>> wrote:<br>
>>>> 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>
>>>><br>
>>>> "was directly invoked by the user. User operations are not permitted to directly update the persistent configuration of a server in a managed domain."<br>
>>>><br>
>>>> This is coming from ControllerLogger#<wbr>modelUpdateNotAuthorized (in wildfly-core) because we define the resource as *not* being runtime only:<br>
>>>><br>
>>>> public boolean isRuntime() {return false;}<br>
>>>><br>
>>>> 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>
>>>><br>
>>>> Thanks,<br>
>>>> Mike<br>
>>>><br>
>>>><br>
>>>> --<br>
>>>> Michael Musgrove<br>
>>>> Transactions Team<br>
>>>> e: <a href="mailto:mmusgrov@redhat.com">mmusgrov@redhat.com</a><br>
>>>><br>
>>>> Our mission:To be the catalyst in communities of customers, contributors, and partners creating better technology the open source way.<br>
>>>><br>
>>>> JBoss, by Red Hat<br>
>>>> Registered Address: Red Hat Ltd, 6700 Cork Airport Business Park, Kinsale Road, Co. Cork.<br>
>>>> Registered in the Companies Registration Office, Parnell House, 14 Parnell Square, Dublin 1, Ireland, No.304873<br>
>>>> Directors:Michael Cunningham (USA), Vicky Wiseman (USA), Michael O'Neill, Keith Phelan, Matt Parson (USA)<br>
>>>><br>
>>>><br>
>>>><br>
>>>> --<br>
>>>> Michael Musgrove<br>
>>>> Transactions Team<br>
>>>> e: <a href="mailto:mmusgrov@redhat.com">mmusgrov@redhat.com</a><br>
>>>><br>
>>>> Our mission:To be the catalyst in communities of customers, contributors, and partners creating better technology the open source way.<br>
>>>><br>
>>>> JBoss, by Red Hat<br>
>>>> Registered Address: Red Hat Ltd, 6700 Cork Airport Business Park, Kinsale Road, Co. Cork.<br>
>>>> Registered in the Companies Registration Office, Parnell House, 14 Parnell Square, Dublin 1, Ireland, No.304873<br>
>>>> Directors:Michael Cunningham (USA), Vicky Wiseman (USA), Michael O'Neill, Keith Phelan, Matt Parson (USA)<br>
>>>> ______________________________<wbr>_________________<br>
>>>> wildfly-dev mailing list<br>
>>>> <a href="mailto:wildfly-dev@lists.jboss.org">wildfly-dev@lists.jboss.org</a><br>
>>>> <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>
>>><br>
>>> --<br>
>>> Brian Stansberry<br>
>>> Manager, Senior Principal Software Engineer<br>
>>> JBoss by Red Hat<br>
>>><br>
>>><br>
>>><br>
>><br>
>> --<br>
>> Brian Stansberry<br>
>> Manager, Senior Principal Software Engineer<br>
>> JBoss by Red Hat<br>
>><br>
>><br>
>><br>
>><br>
>><br>
>><br>
>> --<br>
>> Michael Musgrove<br>
>> Transactions Team<br>
>> e: <a href="mailto:mmusgrov@redhat.com">mmusgrov@redhat.com</a><br>
>><br>
>> Our mission:To be the catalyst in communities of customers, contributors, and partners creating better technology the open source way.<br>
>><br>
>> JBoss, by Red Hat<br>
>> Registered Address: Red Hat Ltd, 6700 Cork Airport Business Park, Kinsale Road, Co. Cork.<br>
>> Registered in the Companies Registration Office, Parnell House, 14 Parnell Square, Dublin 1, Ireland, No.304873<br>
>> Directors:Michael Cunningham (USA), Vicky Wiseman (USA), Michael O'Neill, Keith Phelan, Matt Parson (USA)<br>
><br>
> --<br>
> Brian Stansberry<br>
> Manager, Senior Principal Software Engineer<br>
> JBoss by Red Hat<br>
><br>
><br>
><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'Neill, Keith Phelan, Matt Parson (USA)</div></div></div></div></div></div></div>
</div>