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

	F31882360: D2580.id.diff
	Fri, Jun 5, 04:25

	Unknown Object (File)
	Wed, May 27, 14:18

	Unknown Object (File)
	Mon, May 25, 08:56

	Unknown Object (File)
	Mon, May 25, 00:41

	Unknown Object (File)
	Sun, May 24, 19:48

	Unknown Object (File)
	Sun, May 24, 07:03

	Unknown Object (File)
	Sat, May 23, 10:43

	Unknown Object (File)
	Tue, May 19, 13:21

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