laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/41974?usp=email ) ( 1 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: es9p_client: MAke install notification code execute at all ...................................................................... es9p_client: MAke install notification code execute at all The caller specified 'install' but the do_notification() function compared with 'download' :( Change-Id: I2d441cfbc1457688eb163301d3d91a1f1fdc7a8c --- M contrib/es9p_client.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: dexter: Looks good to me, approved Jenkins Builder: Verified diff --git a/contrib/es9p_client.py b/contrib/es9p_client.py index c1455dc..d1e164f 100755 --- a/contrib/es9p_client.py +++ b/contrib/es9p_client.py @@ -126,7 +126,7 @@ if self.opts.iccid: ntf_metadata['iccid'] = h2b(swap_nibbles(self.opts.iccid)) - if self.opts.operation == 'download': + if self.opts.operation == 'install': pird = { 'transactionId': h2b(self.opts.transaction_id), 'notificationMetadata': ntf_metadata, -- To view, visit https://gerrit.osmocom.org/c/pysim/+/41974?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I2d441cfbc1457688eb163301d3d91a1f1fdc7a8c Gerrit-Change-Number: 41974 Gerrit-PatchSet: 3 Gerrit-Owner: laforge <laforge@osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier@sysmocom.de> Gerrit-Reviewer: laforge <laforge@osmocom.org>