Skip to content
Snippets Groups Projects
Commit 7d687061 authored by Lasse Tjernaes Wardenaer's avatar Lasse Tjernaes Wardenaer
Browse files

Resolve "Remove stoi and switch to function that supports enough bits"

parent 90b67165
No related branches found
No related tags found
No related merge requests found
Pipeline #44197 failed
# v.NEXT
## Summary
### Features
### Bug Fixes
- cta/CTA#234 - Replace stoi with toUint64 in standalone cli tool
# v4.8.2-1 # v4.8.2-1
## Summary ## Summary
......
...@@ -128,7 +128,7 @@ void ChangeStorageClass::updateStorageClassInEosNamespace() { ...@@ -128,7 +128,7 @@ void ChangeStorageClass::updateStorageClassInEosNamespace() {
const auto [diskInstance, diskFileId] = CatalogueFetch::getInstanceAndFid(archiveFileId, m_serviceProviderPtr, m_log); const auto [diskInstance, diskFileId] = CatalogueFetch::getInstanceAndFid(archiveFileId, m_serviceProviderPtr, m_log);
// No files in flight should change storage class // No files in flight should change storage class
const auto md_response = m_endpointMapPtr->getMD(diskInstance, ::eos::rpc::FILE, std::stoi(diskFileId), "", false); const auto md_response = m_endpointMapPtr->getMD(diskInstance, ::eos::rpc::FILE, cta::utils::toUint64(diskFileId), "", false);
if (fileInFlight(md_response.fmd().locations())) { if (fileInFlight(md_response.fmd().locations())) {
m_archiveIdsNotUpdatedInEos.push_back(archiveFileId); m_archiveIdsNotUpdatedInEos.push_back(archiveFileId);
std::list<cta::log::Param> params; std::list<cta::log::Param> params;
......
...@@ -26,6 +26,7 @@ ...@@ -26,6 +26,7 @@
#include "cmdline/standalone_cli_tools/common/CatalogueFetch.hpp" #include "cmdline/standalone_cli_tools/common/CatalogueFetch.hpp"
#include "common/exception/UserError.hpp" #include "common/exception/UserError.hpp"
#include "common/log/StdoutLogger.hpp" #include "common/log/StdoutLogger.hpp"
#include "common/utils/utils.hpp"
#include "cta_frontend.pb.h" //!< Auto-generated message types from .proto file #include "cta_frontend.pb.h" //!< Auto-generated message types from .proto file
#include "version.h" #include "version.h"
...@@ -119,7 +120,7 @@ std::tuple<std::string,std::string> CatalogueFetch::getInstanceAndFid(const std: ...@@ -119,7 +120,7 @@ std::tuple<std::string,std::string> CatalogueFetch::getInstanceAndFid(const std:
admincmd->set_subcmd(cta::admin::AdminCmd::SUBCMD_LS); admincmd->set_subcmd(cta::admin::AdminCmd::SUBCMD_LS);
auto new_opt = admincmd->add_option_uint64(); auto new_opt = admincmd->add_option_uint64();
new_opt->set_key(cta::admin::OptionUInt64::ARCHIVE_FILE_ID); new_opt->set_key(cta::admin::OptionUInt64::ARCHIVE_FILE_ID);
new_opt->set_value(std::stoi(archiveFileId)); new_opt->set_value(cta::utils::toUint64(archiveFileId));
handleResponse(request, serviceProviderPtr); handleResponse(request, serviceProviderPtr);
......
...@@ -237,7 +237,7 @@ void RestoreFilesCmd::listDeletedFilesCta() const { ...@@ -237,7 +237,7 @@ void RestoreFilesCmd::listDeletedFilesCta() const {
auto key = cta::admin::OptionUInt64::ARCHIVE_FILE_ID; auto key = cta::admin::OptionUInt64::ARCHIVE_FILE_ID;
auto new_opt = admincmd.add_option_uint64(); auto new_opt = admincmd.add_option_uint64();
new_opt->set_key(key); new_opt->set_key(key);
new_opt->set_value(std::stoi(m_archiveFileId.value())); new_opt->set_value(cta::utils::toUint64(m_archiveFileId.value()));
} }
if (m_copyNumber) { if (m_copyNumber) {
params.push_back(cta::log::Param("copyNb", m_copyNumber.value())); params.push_back(cta::log::Param("copyNb", m_copyNumber.value()));
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment