Skip to content
Closed
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
feat: add command to get basic storage info
Signed-off-by: Robin Appelman <[email protected]>
  • Loading branch information
icewind1991 authored and AndyScherzinger committed Jun 12, 2025
commit c415c3cacbb307ac18453d5abf974fd828b589f5
97 changes: 97 additions & 0 deletions core/Command/Info/FileUtils.php
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@
use OCA\Files_Sharing\SharedMount;
use OCA\GroupFolders\Mount\GroupMountPoint;
use OCP\Constants;
use OCP\DB\QueryBuilder\IQueryBuilder;
use OCP\Files\Config\IUserMountCache;
use OCP\Files\FileInfo;
use OCP\Files\Folder;
Expand All @@ -21,14 +22,19 @@
use OCP\Files\Mount\IMountPoint;
use OCP\Files\Node;
use OCP\Files\NotFoundException;
use OCP\IDBConnection;
use OCP\Share\IShare;
use OCP\Util;
use Symfony\Component\Console\Output\OutputInterface;

/**
* @psalm-type StorageInfo array{numeric_id: int, id: string, available: bool, last_checked: ?\DateTime, files: int}
*/
class FileUtils {
public function __construct(
private IRootFolder $rootFolder,
private IUserMountCache $userMountCache,
private IDBConnection $connection,
) {
}

Expand Down Expand Up @@ -219,4 +225,95 @@ public function outputLargeFilesTree(
}
return $count;
}

public function getNumericStorageId(string $id): ?int {
if (is_numeric($id)) {
return (int)$id;
}
$query = $this->connection->getQueryBuilder();
$query->select('numeric_id')
->from('storages')
->where($query->expr()->eq('id', $query->createNamedParameter($id)));
$result = $query->executeQuery()->fetchOne();
return $result ? (int)$result : null;

Check notice

Code scanning / Psalm

RiskyTruthyFalsyComparison Note

Operand of type false|mixed contains type mixed, which can be falsy and truthy. This can cause possibly unexpected behavior. Use strict comparison instead.
}

/**
* @param int|null $limit
* @return ?StorageInfo
* @throws \OCP\DB\Exception
*/
public function getStorage(int $id): ?array {
$query = $this->connection->getQueryBuilder();
$query->select('numeric_id', 'id', 'available', 'last_checked')
->selectAlias($query->func()->count('fileid'), 'files')
->from('storages', 's')
->innerJoin('s', 'filecache', 'f', $query->expr()->eq('f.storage', 's.numeric_id'))
->where($query->expr()->eq('s.numeric_id', $query->createNamedParameter($id, IQueryBuilder::PARAM_INT)))
->groupBy('s.numeric_id', 's.id', 's.available', 's.last_checked');
$row = $query->executeQuery()->fetch();
if ($row) {
return [
'numeric_id' => $row['numeric_id'],
'id' => $row['id'],
'files' => $row['files'],
'available' => (bool)$row['available'],
'last_checked' => $row['last_checked'] ? new \DateTime('@' . $row['last_checked']) : null,
];
} else {
return null;
}
}

/**
* @param int|null $limit
* @return \Iterator<StorageInfo>
* @throws \OCP\DB\Exception
*/
public function listStorages(?int $limit): \Iterator {
$query = $this->connection->getQueryBuilder();
$query->select('numeric_id', 'id', 'available', 'last_checked')
->selectAlias($query->func()->count('fileid'), 'files')
->from('storages', 's')
->innerJoin('s', 'filecache', 'f', $query->expr()->eq('f.storage', 's.numeric_id'))
->groupBy('s.numeric_id', 's.id', 's.available', 's.last_checked')
->orderBy('files', 'DESC');
if ($limit !== null) {
$query->setMaxResults($limit);
}
$result = $query->executeQuery();
while ($row = $result->fetch()) {
yield [
'numeric_id' => $row['numeric_id'],
'id' => $row['id'],
'files' => $row['files'],
'available' => (bool)$row['available'],
'last_checked' => $row['last_checked'] ? new \DateTime('@' . $row['last_checked']) : null,
];
}
}

/**
* @param StorageInfo $storage
* @return array
*/
public function formatStorage(array $storage): array {
return [
'numeric_id' => $storage['numeric_id'],
'id' => $storage['id'],
'files' => $storage['files'],
'available' => $storage['available'] ? 'true' : 'false',
'last_checked' => $storage['last_checked']?->format(\DATE_ATOM),
];
}

/**
* @param \Iterator<StorageInfo> $storages
* @return \Iterator
*/
public function formatStorages(\Iterator $storages): \Iterator {
foreach ($storages as $storage) {
yield $this->formatStorage($storage);
}
}
}
49 changes: 49 additions & 0 deletions core/Command/Info/Storage.php
Original file line number Diff line number Diff line change
@@ -0,0 +1,49 @@
<?php

declare(strict_types=1);
/**
* SPDX-FileCopyrightText: 2023 Nextcloud GmbH and Nextcloud contributors
* SPDX-License-Identifier: AGPL-3.0-or-later
*/

namespace OC\Core\Command\Info;

use OC\Core\Command\Base;
use OCP\IDBConnection;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;

class Storage extends Base {
public function __construct(
private readonly IDBConnection $connection,
private readonly FileUtils $fileUtils,
) {
parent::__construct();
}

protected function configure(): void {
parent::configure();
$this
->setName('info:storage')
->setDescription('Get information a single storage')
->addArgument('storage', InputArgument::REQUIRED, 'Storage to get information for');
}

public function execute(InputInterface $input, OutputInterface $output): int {
$storage = $input->getArgument('storage');
$storageId = $this->fileUtils->getNumericStorageId($storage);
if (!$storageId) {

Check notice

Code scanning / Psalm

RiskyTruthyFalsyComparison Note

Operand of type int|null contains type int, which can be falsy and truthy. This can cause possibly unexpected behavior. Use strict comparison instead.
$output->writeln('<error>No storage with id ' . $storage . ' found</error>');
return 1;
}

$info = $this->fileUtils->getStorage($storageId);
if (!$info) {
$output->writeln('<error>No storage with id ' . $storage . ' found</error>');
return 1;
}
$this->writeArrayInOutputFormat($input, $output, $this->fileUtils->formatStorage($info));
return 0;
}
}
43 changes: 43 additions & 0 deletions core/Command/Info/Storages.php
Original file line number Diff line number Diff line change
@@ -0,0 +1,43 @@
<?php

declare(strict_types=1);
/**
* SPDX-FileCopyrightText: 2023 Nextcloud GmbH and Nextcloud contributors
* SPDX-License-Identifier: AGPL-3.0-or-later
*/

namespace OC\Core\Command\Info;

use OC\Core\Command\Base;
use OCP\IDBConnection;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Output\OutputInterface;

class Storages extends Base {
public function __construct(
private readonly IDBConnection $connection,
private readonly FileUtils $fileUtils,
) {
parent::__construct();
}

protected function configure(): void {
parent::configure();
$this
->setName('info:storages')
->setDescription('List storages ordered by the number of files')
->addOption('count', 'c', InputOption::VALUE_REQUIRED, 'Number of storages to display', 25)
->addOption('all', 'a', InputOption::VALUE_NONE, 'Display all storages');
}

public function execute(InputInterface $input, OutputInterface $output): int {
$count = (int)$input->getOption('count');
$all = $input->getOption('all');

$limit = $all ? null : $count;
$storages = $this->fileUtils->listStorages($limit);
$this->writeStreamingTableInOutputFormat($input, $output, $this->fileUtils->formatStorages($storages), 100);
return 0;
}
}
2 changes: 2 additions & 0 deletions core/register_command.php
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,8 @@

$application->add(Server::get(Command\Info\File::class));
$application->add(Server::get(Command\Info\Space::class));
$application->add(Server::get(Command\Info\Storage::class));
$application->add(Server::get(Command\Info\Storages::class));

$application->add(Server::get(Command\Db\ConvertType::class));
$application->add(Server::get(Command\Db\ConvertMysqlToMB4::class));
Expand Down
2 changes: 2 additions & 0 deletions lib/composer/composer/autoload_classmap.php
Original file line number Diff line number Diff line change
Expand Up @@ -1195,6 +1195,8 @@
'OC\\Core\\Command\\Info\\File' => $baseDir . '/core/Command/Info/File.php',
'OC\\Core\\Command\\Info\\FileUtils' => $baseDir . '/core/Command/Info/FileUtils.php',
'OC\\Core\\Command\\Info\\Space' => $baseDir . '/core/Command/Info/Space.php',
'OC\\Core\\Command\\Info\\Storage' => $baseDir . '/core/Command/Info/Storage.php',
'OC\\Core\\Command\\Info\\Storages' => $baseDir . '/core/Command/Info/Storages.php',
'OC\\Core\\Command\\Integrity\\CheckApp' => $baseDir . '/core/Command/Integrity/CheckApp.php',
'OC\\Core\\Command\\Integrity\\CheckCore' => $baseDir . '/core/Command/Integrity/CheckCore.php',
'OC\\Core\\Command\\Integrity\\SignApp' => $baseDir . '/core/Command/Integrity/SignApp.php',
Expand Down
2 changes: 2 additions & 0 deletions lib/composer/composer/autoload_static.php
Original file line number Diff line number Diff line change
Expand Up @@ -1228,6 +1228,8 @@ class ComposerStaticInit749170dad3f5e7f9ca158f5a9f04f6a2
'OC\\Core\\Command\\Info\\File' => __DIR__ . '/../../..' . '/core/Command/Info/File.php',
'OC\\Core\\Command\\Info\\FileUtils' => __DIR__ . '/../../..' . '/core/Command/Info/FileUtils.php',
'OC\\Core\\Command\\Info\\Space' => __DIR__ . '/../../..' . '/core/Command/Info/Space.php',
'OC\\Core\\Command\\Info\\Storage' => __DIR__ . '/../../..' . '/core/Command/Info/Storage.php',
'OC\\Core\\Command\\Info\\Storages' => __DIR__ . '/../../..' . '/core/Command/Info/Storages.php',
'OC\\Core\\Command\\Integrity\\CheckApp' => __DIR__ . '/../../..' . '/core/Command/Integrity/CheckApp.php',
'OC\\Core\\Command\\Integrity\\CheckCore' => __DIR__ . '/../../..' . '/core/Command/Integrity/CheckCore.php',
'OC\\Core\\Command\\Integrity\\SignApp' => __DIR__ . '/../../..' . '/core/Command/Integrity/SignApp.php',
Expand Down