Skip to content

Error message cause should not throw internal error message #8851

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
ramya-c3 opened this issue Mar 5, 2025 · 3 comments · Fixed by #8867
Closed

Error message cause should not throw internal error message #8851

ramya-c3 opened this issue Mar 5, 2025 · 3 comments · Fixed by #8867
Assignees

Comments

@ramya-c3
Copy link

ramya-c3 commented Mar 5, 2025

Environment info

  • NooBaa Version: noobaa-core-5.18.0-20250222.el9.x86_66
  • Platform: Standalone

Actual behavior

  1. noobaa-cli upgrade start --expected_version 5.13.3 2>/dev/null

{
"error": {
"code": "UpgradeFailed",
"message": "Upgrade request failed",
"cause": "Error: system does not exist\n at NCUpgradeManager.upgrade_config_dir (/usr/local/noobaa-core/src/upgrade/nc_upgrade_manager.js:102:33)\n at async start_config_dir_upgrade (/usr/local/noobaa-core/src/manage_nsfs/upgrade.js:55:29)\n at async Object.manage_upgrade_operations (/usr/local/noobaa-core/src/manage_nsfs/upgrade.js:23:13)\n at async main (/usr/local/noobaa-core/src/cmd/manage_nsfs.js:74:13)"
}
}

Expected behavior

noobaa-cli upgrade start --expected_version 5.13.3 2>/dev/null

{
"error": {
"code": "UpgradeFailed",
"message": "Upgrade request failed",
"cause": "The version provided is lower than the installed version"
}
}

Steps to reproduce

More information - Screenshots / Logs / Other output

@ramya-c3 ramya-c3 added the NS-FS label Mar 5, 2025
@shirady shirady self-assigned this Mar 18, 2025
@shirady
Copy link
Contributor

shirady commented Mar 18, 2025

@ramya-c3 did you notice that the error "system does not exist",it is not related to the version?
We couldn't get the system config data.
Could please you retest it with the system config data?

@ramya-c3
Copy link
Author

@shirady Will test and get back on this

@shirady
Copy link
Contributor

shirady commented Apr 1, 2025

@ramya-c3 we added checks with what we understand, if there is still something that you want to raise please open an issue with all the details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants