Paths

Table of Contentst

Rename jails.list into jails.list_jails
ClosedPublic
Actions

Authored by dereckson on Mar 7 2022, 22:13.

Tags

None

Referenced Files

	F26031561: D2580.id6516.diff
	Sun, Apr 19, 17:16

	Unknown Object (File)
	Wed, Apr 8, 17:50

	Unknown Object (File)
	Wed, Apr 8, 09:34

	Unknown Object (File)
	Mon, Apr 6, 05:57

	Unknown Object (File)
	Mon, Apr 6, 04:54

	Unknown Object (File)
	Sun, Apr 5, 20:05

	Unknown Object (File)
	Sun, Apr 5, 19:39

	Unknown Object (File)
	Wed, Apr 1, 20:01

View All Files

Subscribers

None

Details

Reviewers

dereckson

Commits

rOPSe89c73d4163b: Rename jails.list into jails.list_jails

Summary

list() is a reserved method name in Python,
so it shadows it.

This is a follow-up for 5a1d179f76e5.

Test Plan

make test

Diff Detail

Repository

rOPS Nasqueron Operations

Lint

Lint Passed

Unit

No Test Coverage

Branch

fix-shadow-jails

Build Status

Buildable 4041
Build 4293: arc lint + arc unit

Event Timeline

dereckson requested review of this revision.Mar 7 2022, 22:13

dereckson created this revision.

Harbormaster completed remote builds in B4041: Diff 6516.Mar 7 2022, 22:13

dereckson accepted this revision.Mar 7 2022, 22:13

This revision is now accepted and ready to land.Mar 7 2022, 22:13

Closed by commit rOPSe89c73d4163b: Rename jails.list into jails.list_jails (authored by dereckson). · Explain WhyMar 7 2022, 22:14

This revision was automatically updated to reflect the committed changes.

dereckson added a commit: rOPSe89c73d4163b: Rename jails.list into jails.list_jails.

Revision Contents
Changeset List

Path

Size

_tests/

modules/

test_jails.py

6 lines

Diff 6516

View Options

_tests/modules/test_jails.py

Rename jails.list into jails.list_jailsClosedPublicActions