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
Password field in loading XML files can be empty #221
Labels
alert: NEED ACCOUNT KEY
Need to assign an account key to this issue
component: load
Tasks related to the database loading
priority: blocker
Blocker
requestor: NOAA/EMC
NOAA Environmental Modeling Center
type: bug
Fix something that is not working
Milestone
Comments
TatianaBurek
added
alert: NEED ACCOUNT KEY
Need to assign an account key to this issue
component: load
Tasks related to the database loading
priority: blocker
Blocker
type: bug
Fix something that is not working
alert: NEED CYCLE ASSIGNMENT
Need to assign to a release development cycle
labels
Aug 7, 2023
TatianaBurek
changed the title
Password field in loading/batch XML files can be empty
Password field in loading XML files can be empty
Aug 7, 2023
JohnHalleyGotway
removed
the
alert: NEED CYCLE ASSIGNMENT
Need to assign to a release development cycle
label
Aug 7, 2023
@TatianaBurek , should we still keep this check, except log a WARNING so non-cloud users still get notification of a missing password? (And continue without erroring out) |
bikegeek
added a commit
that referenced
this issue
Aug 7, 2023
…rd is missing. This allows cloud-users to continue without a password.
bikegeek
added a commit
that referenced
this issue
Aug 7, 2023
…ord. If the username is missing, continue to log an error and raise a NameError. However, if the password is missing, log a warning and continue, to support cloud-based instances.
I don't think so. |
bikegeek
added a commit
that referenced
this issue
Aug 7, 2023
14 tasks
bikegeek
added a commit
that referenced
this issue
Aug 16, 2023
* Issue #221 Replace the ERROR with a WARNING and proceed when a password is missing. This allows cloud-users to continue without a password. * Issue #221 Do a separate check, one for username, the other for password. If the username is missing, continue to log an error and raise a NameError. However, if the password is missing, log a warning and continue, to support cloud-based instances. * Fixed extraneous parenthesis in if-statement. * Issue #221 Removed check for missing password. * update warning message, and verified that the check for password no longer exists
bikegeek
added a commit
that referenced
this issue
Aug 16, 2023
* Issue #221 Replace the ERROR with a WARNING and proceed when a password is missing. This allows cloud-users to continue without a password. * Issue #221 Do a separate check, one for username, the other for password. If the username is missing, continue to log an error and raise a NameError. However, if the password is missing, log a warning and continue, to support cloud-based instances. * Fixed extraneous parenthesis in if-statement. * Issue #221 Removed check for missing password. * update warning message, and verified that the check for password no longer exists
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
alert: NEED ACCOUNT KEY
Need to assign an account key to this issue
component: load
Tasks related to the database loading
priority: blocker
Blocker
requestor: NOAA/EMC
NOAA Environmental Modeling Center
type: bug
Fix something that is not working
Describe the Problem
This issue arose via METplus Discussions dtcenter/METplus#2291.
Currently if the password field in loading XML file is empty,
METdbLoad script throws an error:
METdataio/METdbLoad/ush/read_load_xml.py
Line 242 in 2c35b04
In reality, password field can be empty - example the environment on AWS.
Because of this restriction, users can't load their data to AWS databases.
Recommend that Binbin Zhou and Ho-Chun Huang test and verify this fix.
Expected Behavior
The check the empty password field should be removed
Environment
Describe your runtime environment:
1. Machine: (e.g. HPC name, Linux Workstation, Mac Laptop)
2. OS: (e.g. RedHat Linux, MacOS)
3. Software version number(s)
To Reproduce
Describe the steps to reproduce the behavior:
1. Go to '...'
2. Click on '....'
3. Scroll down to '....'
4. See error
Post relevant sample data following these instructions:
https://dtcenter.org/community-code/model-evaluation-tools-met/met-help-desk#ftp
Relevant Deadlines
Needs to be fixed, included in a 2.1.1 bugfix release, and redeployed on NOAA AWS no later than August 29, 2023.
This is critical for NOAA AQM plotting.
Funding Source
Define the source of funding and account keys here or state NONE.
Define the Metadata
Assignee
Labels
Projects and Milestone
Define Related Issue(s)
Consider the impact to the other METplus components.
Bugfix Checklist
See the METplus Workflow for details.
Branch name:
bugfix_<Issue Number>_main_<Version>_<Description>
Pull request:
bugfix <Issue Number> main_<Version> <Description>
Select: Reviewer(s) and Development issues
Select: Organization level software support Project for the current coordinated release
Select: Milestone as the next bugfix version
Branch name:
bugfix_<Issue Number>_develop_<Description>
Pull request:
bugfix <Issue Number> develop <Description>
Select: Reviewer(s) and Development issues
Select: Repository level development cycle Project for the next official release
Select: Milestone as the next official version
The text was updated successfully, but these errors were encountered: