Bug#982682: ansible: some syntax errors appear in the installation

2021-03-09 Thread Lee Garrett
On 09/03/2021 06:38, Louis-Philippe Véronneau wrote:
> I can confirm I can reproduce this bug. Had the same error message while
> upgrading to ansible 2.10.7-1 on a sid machine.
> 

Thanks for the update! I'll try to fix it with the next upload, however
I'm currently waiting for an unblock request to go through.



Bug#982682: ansible: some syntax errors appear in the installation

2021-03-08 Thread Louis-Philippe Véronneau
I can confirm I can reproduce this bug. Had the same error message while
upgrading to ansible 2.10.7-1 on a sid machine.

-- 
  ⢀⣴⠾⠻⢶⣦⠀
  ⣾⠁⢠⠒⠀⣿⡁  Louis-Philippe Véronneau
  ⢿⡄⠘⠷⠚⠋   po...@debian.org / veronneau.org
  ⠈⠳⣄



OpenPGP_signature
Description: OpenPGP digital signature


Bug#982682: ansible: some syntax errors appear in the installation

2021-02-13 Thread nozzy123nozzy
Package: ansible
Version: 2.10.7-1
Severity: normal

Dear Maintainer,

I installed the ansible package into my Debian box, and I got errors as
follows,

---cut--here--
$ env LANG=C sudo apt install ansible
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
Suggested packages:
  cowsay sshpass
The following NEW packages will be installed:
  ansible
0 upgraded, 1 newly installed, 0 to remove and 0 not upgraded.
Need to get 16.8 MB of archives.
After this operation, 198 MB of additional disk space will be used.
Get:1 http://deb.debian.org/debian sid/main amd64 ansible all 2.10.7-1
[16.8 MB]
Fetched 16.8 MB in 12s (1389 kB/s)
Selecting previously unselected package ansible.
(Reading database ... 349484 files and directories currently
installed.)
Preparing to unpack .../ansible_2.10.7-1_all.deb ...
Unpacking ansible (2.10.7-1) ...
Setting up ansible (2.10.7-1) ...
/usr/lib/python3/dist-
packages/ansible_collections/azure/azcollection/plugins/modules/azure_r
m_registrationassignment.py:233: SyntaxWarning: "is" with a literal.
Did you mean "=="?
  if self.state is 'present':
/usr/lib/python3/dist-
packages/ansible_collections/cyberark/pas/plugins/modules/
cyberark_account.py:532: SyntaxWarning: 'str' object is not callable;
perhaps you missed a comma?
  ("child_module_parm_value: %s "
/usr/lib/python3/dist-
packages/ansible_collections/cyberark/pas/plugins/modules/
cyberark_account.py:573: SyntaxWarning: 'str' object is not callable;
perhaps you missed a comma?
  "parameter_name=%s  value=%s existing=%s"
/usr/lib/python3/dist-
packages/ansible_collections/cyberark/pas/plugins/modules/
cyberark_account.py:634: SyntaxWarning: 'str' object is not callable;
perhaps you missed a comma?
  "parameter_name=%s  value=%s existing=%s"
Scanning processes...
Scanning processor microcode...
Scanning linux images...
---cut-here--

Then,

Then,

Q1.I attach the patch in this mail to fix this problem. 
Could you apply this?

Q2. I'm looking for source code of upstream related to this error. Does
someone know where is upstream of the following two source code?
 - ansible_collections/azure/azcollection/plugins/mo
dules/azure_rm_registrationassignment.py
 - ansible_collections/cyberark/pas/plugins/modules/
cyberark_account.py

I tried to find out the upstreams of these codes, but I don't find them
yet.

I really appreciate any help you can provide.

Takahide Nojima 

-- System Information:
Debian Release: bullseye/sid
  APT prefers unstable-debug
  APT policy: (500, 'unstable-debug'), (500, 'unstable'), (500,
'testing')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 5.10.0-3-amd64 (SMP w/4 CPU threads)
Locale: LANG=ja_JP.UTF-8, LC_CTYPE=ja_JP.UTF-8 (charmap=UTF-8),
LANGUAGE not set
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)
LSM: AppArmor: enabled

Versions of packages ansible depends on:
ii  ansible-base   2.10.5+dfsg-1
ii  openssh-client 1:8.4p1-3
ii  python33.9.1-1
ii  python3-distutils  3.9.1-2
ii  python3-dnspython  2.0.0-1
ii  python3-httplib2   0.18.1-3
ii  python3-jinja2 2.11.2-1
ii  python3-netaddr0.7.19-4
ii  python3-yaml   5.3.1-3+b1

Versions of packages ansible recommends:
ii  python3-argcomplete   1.8.1-1.4
ii  python3-cryptography  3.3.2-1
ii  python3-jmespath  0.10.0-1
ii  python3-kerberos  1.1.14-3.1+b3
ii  python3-libcloud  3.2.0-2
ii  python3-selinux   3.1-3
ii  python3-winrm 0.3.0-2
ii  python3-xmltodict 0.12.0-2

Versions of packages ansible suggests:
pn  cowsay   
pn  sshpass  

-- no debconf information
diff -uNr org/debian/changelog new/debian/changelog
--- org/debian/changelog	2021-02-10 08:44:56.0 +0900
+++ new/debian/changelog	2021-02-13 17:01:55.768549872 +0900
@@ -1,3 +1,10 @@
+ansible (2.10.7-1.1) UNRELEASED; urgency=medium
+
+  * fix some syntax errors in azure_rm_registrationassignment.py
+  * fix some syntax errors in cyberark_account.py 
+
+ -- Takahide Nojima   Sat, 13 Feb 2021 16:59:58 +0900
+
 ansible (2.10.7-1) unstable; urgency=medium
 
   * New upstream release.
diff -uNr org/debian/patches/fix-subttle-syntax-errors.patch new/debian/patches/fix-subttle-syntax-errors.patch
--- org/debian/patches/fix-subttle-syntax-errors.patch	1970-01-01 09:00:00.0 +0900
+++ new/debian/patches/fix-subttle-syntax-errors.patch	2021-02-13 16:38:43.110158396 +0900
@@ -0,0 +1,40 @@
+--- a/ansible_collections/azure/azcollection/plugins/modules/azure_rm_registrationassignment.py
 b/ansible_collections/azure/azcollection/plugins/modules/azure_rm_registrationassignment.py
+@@ -230,7 +230,7 @@
+ response = old_response
+ self.results['state'] = response
+ 
+-if self.state is 'present':
++if self.state == 'present':
+ if self.results['state'].get('properties', None) is not None:
+