Page MenuHomePhabricator

Upgrade various Cloud VPS Python 2 scripts to Python 3
Open, MediumPublic

Description

So based on a git grep -E "^#\!/usr/bin/(env )?python([^3]|$)" in puppet.git I filtered through for stuff that looked Cloud VPS-related. It might be incomplete.
Ideally this stuff would be done this year.

modules/openstack/files/designate/designatemakedomain.py:#!/usr/bin/python
modules/openstack/files/monitor/fullstack/check_nova_fullstack_leaks.py:#!/usr/bin/python
modules/openstack/files/ussuri/admin_scripts/wmcs-cold-migrate.py:#!/usr/bin/python
modules/openstack/files/ussuri/admin_scripts/wmcs-cold-nova-migrate.py:#!/usr/bin/python
modules/openstack/files/ussuri/admin_scripts/wmcs-live-migrate.py:#!/usr/bin/python
modules/openstack/files/ussuri/admin_scripts/wmcs-logstat.py:#!/usr/bin/env python
modules/openstack/files/ussuri/admin_scripts/wmcs-makedomain.py:#!/usr/bin/python
modules/openstack/files/ussuri/admin_scripts/wmcs-region-migrate-security-groups.py:#!/usr/bin/python
modules/openstack/files/ussuri/admin_scripts/wmcs-region-migrate.py:#!/usr/bin/python
modules/openstack/files/ussuri/admin_scripts/wmcs-wikitech-grep.py:#!/usr/bin/env python
modules/openstack/files/victoria/admin_scripts/wmcs-cold-migrate.py:#!/usr/bin/python
modules/openstack/files/victoria/admin_scripts/wmcs-cold-nova-migrate.py:#!/usr/bin/python
modules/openstack/files/victoria/admin_scripts/wmcs-live-migrate.py:#!/usr/bin/python
modules/openstack/files/victoria/admin_scripts/wmcs-logstat.py:#!/usr/bin/env python
modules/openstack/files/victoria/admin_scripts/wmcs-makedomain.py:#!/usr/bin/python
modules/openstack/files/victoria/admin_scripts/wmcs-region-migrate-security-groups.py:#!/usr/bin/python
modules/openstack/files/victoria/admin_scripts/wmcs-region-migrate.py:#!/usr/bin/python
modules/openstack/files/victoria/admin_scripts/wmcs-wikitech-grep.py:#!/usr/bin/env python
modules/profile/files/toolforge/proxylistener.py:#!/usr/bin/env python

Related Objects

StatusSubtypeAssignedTask
InvalidNone
OpenNone
ResolvedBUG REPORT Bstorm
ResolvedMoritzMuehlenhoff
ResolvedNone
ResolvedEevans
ResolvedKrenair
ResolvedKrenair
Resolvedelukey
ResolvedNone
DeclinedNone
DeclinedNone
ResolvedNone
ResolvedJdforrester-WMF
ResolvedKrenair
Resolvedthcipriani
Resolvedtaavi
Resolvedtaavi
Resolvedhashar
Resolvedtaavi
Resolvedtaavi
Resolvedtaavi
ResolvedAndrew
ResolvedMusikAnimal
ResolvedMusikAnimal
ResolvedAndrew
DuplicateNone
Resolved Phamhi
ResolvedAndrew
Resolved Bstorm
Resolved Bstorm
Resolved Bstorm

Event Timeline

The following:

modules/base/files/labs/notify_maintainers.py
modules/base/files/labs/puppet_alert.py
modules/ldap/files/scripts/ssh-key-ldap-lookup
modules/shinken/files/shinkengen

are probably blocked on T218423, probably.

bd808 renamed this task from Upgrade various Python 2 scripts to Python 3 to Upgrade various Cloud VPS/Toolforge Python 2 scripts to Python 3 .Mar 15 2019, 7:36 PM

@bd808 I actually separated the toolforge scripts into a different list which I haven't opened a task for yet

Krenair renamed this task from Upgrade various Cloud VPS/Toolforge Python 2 scripts to Python 3 to Upgrade various Cloud VPS Python 2 scripts to Python 3 .Mar 15 2019, 7:40 PM

I think some of this stuff like dynamicproxy/invisible-unicorn.py would be easier to port/test if they were in separate repos - any opinions on that?

I think some of this stuff like dynamicproxy/invisible-unicorn.py would be easier to port/test if they were in separate repos - any opinions on that?

Our infrastructure for building and deploying deb packages is horrible. I would really not be excited about making dynamicproxy harder to deploy.

I think some of this stuff like dynamicproxy/invisible-unicorn.py would be easier to port/test if they were in separate repos - any opinions on that?

Our infrastructure for building and deploying deb packages is horrible. I would really not be excited about making dynamicproxy harder to deploy.

Oh, agreed. I was thinking of deploying out of a git checkout or something. But I'll try adding some tests to the existing puppet repo and see how it goes.

Change 622844 had a related patch set uploaded (by Bstorm; owner: Bstorm):
[operations/puppet@production] remove python2-only function from puppet_alert to move to py3

https://gerrit.wikimedia.org/r/622844

Change 622846 had a related patch set uploaded (by Bstorm; owner: Bstorm):
[operations/puppet@production] move puppet_alert script to python3

https://gerrit.wikimedia.org/r/622846

Change 622874 had a related patch set uploaded (by Bstorm; owner: Bstorm):
[operations/puppet@production] cloud-vps: Add python3 client packages in cloud

https://gerrit.wikimedia.org/r/622874

Change 622874 merged by Bstorm:
[operations/puppet@production] cloud-vps: Add python3 client packages in cloud

https://gerrit.wikimedia.org/r/622874

Change 622844 merged by Bstorm:
[operations/puppet@production] remove python2-only function from puppet_alert to move to py3

https://gerrit.wikimedia.org/r/622844

Change 624805 had a related patch set uploaded (by Nskaggs; owner: Nskaggs):
[operations/puppet@production] Convert wmcs-novastats scripts to python3

https://gerrit.wikimedia.org/r/624805

Change 622846 merged by Bstorm:
[operations/puppet@production] move puppet_alert script to python3

https://gerrit.wikimedia.org/r/622846

Change 624805 merged by Andrew Bogott:
[operations/puppet@production] Convert wmcs-novastats scripts to python3

https://gerrit.wikimedia.org/r/624805

Change 626235 had a related patch set uploaded (by Nskaggs; owner: Nskaggs):
[operations/puppet@production] Convert maintain-meta_p.py to python3

https://gerrit.wikimedia.org/r/626235

Change 626235 abandoned by Nskaggs:
[operations/puppet@production] Convert maintain-meta_p.py to python3

Reason:
https://gerrit.wikimedia.org/r/c/operations/puppet/ /643363

https://gerrit.wikimedia.org/r/626235

taavi renamed this task from Upgrade various Cloud VPS Python 2 scripts to Python 3 to Upgrade various Cloud VPS Python 2 scripts to Python 3.Oct 28 2021, 6:51 AM

Change 786274 had a related patch set uploaded (by David Caro; author: David Caro):

[operations/puppet@production] openstack: remove ussuri files

https://gerrit.wikimedia.org/r/786274

Change 786274 merged by David Caro:

[operations/puppet@production] openstack: remove ussuri files

https://gerrit.wikimedia.org/r/786274