Ceph

The Ceph C++ code does process these as expected:

test.py:

import glob
import os
import socket
import sqlite3
import struct
import sys
import time
from contextlib import closing

class AdminSocket:
    #SO_PEERCRED = 17 # Pulled from /usr/include/asm-generic/socket.h

    def __init__(self, path):
        self.path = path
        self.getid()

    def getid(self):
        with closing(socket.socket(socket.AF_UNIX, socket.SOCK_STREAM)) as sock:
            sock.connect(self.path)
            creds = sock.getsockopt(socket.SOL_SOCKET, socket.SO_PEERCRED, struct.calcsize('3i'))
            self.pid, self.uid, self.gid = struct.unpack('3i',creds)
            # Do something so we don't see an error: "AdminSocket: error reading request code: (0) Success" 
            sock.sendall(b"{\"prefix\": \"help\", \"format\": \"json\"}\x00")
            length = struct.unpack('>i', sock.recv(4))[0]
            sock.recv(length, socket.MSG_WAITALL)

    def cmd(self, J):
        J = J.encode('utf-8') + b'\x00'
        with closing(socket.socket(socket.AF_UNIX, socket.SOCK_STREAM)) as sock:
            sock.connect(self.path)
            sock.sendall(J)
            length = struct.unpack('>i', sock.recv(4))[0]
            return sock.recv(length, socket.MSG_WAITALL)

a = AdminSocket("/tmp/ceph-asok.jxTcHk/mds.c.asok")
print(a.cmd('{"prefix":"status", "format":"json"}'))
print(a.cmd('{"prefix":"scrub start", "path":"/", "scrubops":["force","recursive"], "format":"json"}'))

pdonnell@vossi04 ~/ceph/build$ python3 ~/test.py
b'{"cluster_fsid":"88c3e7d7-ca67-4ff5-91f9-5dc39f28bd2f","whoami":0,"id":4268,"want_state":"up:active","state":"up:active","fs_name":"a","rank_uptime":5933.0511824089999,"mdsmap_epoch":8,"osdmap_epoch":23,"osdmap_epoch_barrier":23,"uptime":5941.7142224130002}'
b'{"return_code":0,"scrub_tag":"051fa890-d2b3-43e3-9354-12b8b33ca6ce","mode":"asynchronous"}'

Please QA with: https://github.com/ceph/ceph/pull/41771

I was wrong about this bug being in Pacific: the old syntax works:

pdonnell@vossi04 ~/ceph/build$ bin/ceph tell mds.c scrub start / force recursive tag
*** DEVELOPER MODE: setting PATH, PYTHONPATH and LD_LIBRARY_PATH ***
2021-06-09T03:00:49.022+0000 7f40686c1700 -1 WARNING: all dangerous and experimental features are enabled.
2021-06-09T03:00:49.061+0000 7f40686c1700 -1 WARNING: all dangerous and experimental features are enabled.
2021-06-09T03:00:49.129+0000 7f402cff9700  0 client.4386 ms_handle_reset on v2:127.0.0.1:6830/1517164866
2021-06-09T03:00:49.241+0000 7f402cff9700  0 client.4393 ms_handle_reset on v2:127.0.0.1:6830/1517164866
{   
    "return_code": 0,
    "scrub_tag": "tag",
    "mode": "asynchronous" 
}

(Did we intend to break the old syntax with the recent positional argument changes?)