Paths

Table of Contentst

Use jdiff instead of jsondiff
ClosedPublic
Actions

Authored by dereckson on Feb 26 2026, 00:17.

Tags

None

Referenced Files

	F34970752: D3978.id10320.diff
	Sat, Jul 4, 17:11

	F34969102: D3978.diff
	Sat, Jul 4, 14:58

	F34899424: D3978.diff
	Fri, Jul 3, 19:07

	F34899420: D3978.diff
	Fri, Jul 3, 19:07

	Unknown Object (File)
	Tue, Jun 30, 23:39

	Unknown Object (File)
	Mon, Jun 22, 22:06

	Unknown Object (File)
	Mon, Jun 22, 15:16

	Unknown Object (File)
	Sun, Jun 21, 17:18

View All Files

Subscribers

None

Details

Reviewers

yousra

Commits

rOPS48e86fa47a4d: Use jdiff instead of jsondiff

Summary

jdiff seems the new name of jsondiff executable.

This fixes the following issue:

$ make test
[...]
run`'s command `jsondiff /tmp/tmp.CXo1sQNPui data/acmedns-merged.json`
exited with code 127, indicating 'Command not found'.

Diff Detail

Repository

rOPS Nasqueron Operations

Lint

Lint Passed

Unit

No Test Coverage

Branch

jdiff

Build Status

Buildable 6371
Build 6655: arc lint + arc unit

Event Timeline

dereckson requested review of this revision.Feb 26 2026, 00:17

dereckson created this revision.

Harbormaster completed remote builds in B6371: Diff 10320.Feb 26 2026, 00:17

Caught this issue during dependencies test.

yousra accepted this revision.Mar 2 2026, 11:45

This revision is now accepted and ready to land.Mar 2 2026, 11:45

Closed by commit rOPS48e86fa47a4d: Use jdiff instead of jsondiff (authored by dereckson, committed by yousra). · Explain WhyMar 2 2026, 11:46

This revision was automatically updated to reflect the committed changes.

yousra added a commit: rOPS48e86fa47a4d: Use jdiff instead of jsondiff.

Revision Contents
Changeset List

Path

Size

_tests/

scripts/

bats/

test_edit_acme_dns_accounts.sh

2 lines

Diff 10320

View Options

_tests/scripts/bats/test_edit_acme_dns_accounts.sh

Use jdiff instead of jsondiffClosedPublicActions