Paths

Table of Contentst

Define commands as a dictionary instead of a tuple
ClosedPublic
Actions

Authored by dereckson on Sep 4 2021, 20:15.

Tags

None

Referenced Files

	F21078101: D2406.id6048.diff
	Thu, Jan 15, 05:17

	F21073714: D2406.id6048.diff
	Thu, Jan 15, 04:02

	F21073710: D2406.id6049.diff
	Thu, Jan 15, 04:02

	F21063454: D2406.diff
	Thu, Jan 15, 01:16

	F21063452: D2406.id6049.diff
	Thu, Jan 15, 01:16

	Unknown Object (File)
	Mon, Jan 12, 07:53

	Unknown Object (File)
	Fri, Jan 2, 15:33

	Unknown Object (File)
	Fri, Jan 2, 13:58

View All Files

Subscribers

None

Details

Reviewers

dereckson

Maniphest Tasks

Unknown Object (Maniphest Task)

Commits

rOPS4509fe3de0a0: Define commands as a dictionary instead of a tuple

Summary

The extraneous comma made commands parsed as a tuple,
not as intended as a dictionary.

This caused any issue to fail the following test:
<command> in commands, as the tuple doesn't have named keys.

Ref T1647.

Test Plan

bats scripts/bats/test_edit_acme_dns_accounts.sh

Diff Detail

Repository

rOPS Nasqueron Operations

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dereckson requested review of this revision.Sep 4 2021, 20:15

dereckson created this revision.

Harbormaster completed remote builds in B3732: Diff 6048.Sep 4 2021, 20:15

dereckson accepted this revision.Sep 4 2021, 20:15

This revision is now accepted and ready to land.Sep 4 2021, 20:15

This revision was landed with ongoing or failed builds.Sep 4 2021, 20:15

Closed by commit rOPS4509fe3de0a0: Define commands as a dictionary instead of a tuple (authored by dereckson). · Explain Why

This revision was automatically updated to reflect the committed changes.

dereckson added a commit: rOPS4509fe3de0a0: Define commands as a dictionary instead of a tuple.

Revision Contents
Changeset List

Path

Size

roles/

paas-docker/

letsencrypt/

files/

edit-acme-dns-accounts.py

2 lines

Diff 6049

View Options

roles/paas-docker/letsencrypt/files/edit-acme-dns-accounts.py

Define commands as a dictionary instead of a tupleClosedPublicActions