Project

General

Profile

Bug #69058

Bug fix for S3 validation

Added by Waqar Ahmed 6 months ago. Updated 5 months ago.

Status:
Ready for Testing
Priority:
No priority
Assignee:
Waqar Ahmed
Category:
Middleware
Target version:
Severity:
Low Medium
Reason for Closing:
Reason for Blocked:
Needs QA:
Yes
Needs Doc:
No
Needs Merging:
No
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:
ChangeLog Required:
No

Description

When the S3 model hasn't been updated in any way, first time we try to update the service partially, that raises exceptions as some values are None at that point in db and we don't handle that appropriately

Risk
We risk breaking s3 validation

Acceptance Criteria
It should be ensured that S3 service starts as desired and we are able to change the fields as we want to even partially. It should also be tried on a system where we don't have s3 configured that partial updates to the service don't raise exceptions.
S3 row can be deleted from db via following command if such a machine is hard to obtain

midclt call datastore.delete 'services.s3' '1'

1 reflects the id of the s3 row. It can be different if it has been manipulated earlier

Associated revisions

Revision dd728c98 (diff)
Added by Waqar Ahmed 6 months ago

Bug fix for S3 validation

This commit fixes an issue where we did not take into account the default db values while validating the S3 config service update fields causing unintended consequences.
Ticket: #69058

Revision 885b2731 (diff)
Added by Waqar Ahmed 6 months ago

Bug fix for S3 validation

This commit fixes an issue where we did not take into account the default db values while validating the S3 config service update fields causing unintended consequences.
Ticket: #69058

History

#1 Updated by Bug Clerk 6 months ago

  • Status changed from Not Started to In Progress

#2 Updated by Bug Clerk 6 months ago

  • Status changed from In Progress to Ready for Testing

#3 Updated by Dru Lavigne 6 months ago

  • Subject changed from S3 validation throws exceptions to Bug fix for S3 validation
  • Needs Doc changed from Yes to No
  • Needs Merging changed from Yes to No

#4 Updated by Dru Lavigne 6 months ago

  • Target version changed from 11.3 to 11.3-BETA1

Also available in: Atom PDF