Project

General

Profile

Bug #71635

Replace prefix appropriately for datastore service in API

Added by Eric Turgeon almost 2 years ago. Updated almost 2 years ago.

Status:
Done
Priority:
No priority
Assignee:
Waqar Ahmed
Category:
Middleware
Seen in:
Severity:
New
Reason for Closing:
Reason for Blocked:
Needs QA:
No
Needs Doc:
No
Needs Merging:
No
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:
ChangeLog Required:
No

Description

API test error: https://builds.ixsystems.com/jenkins/job/FreeNAS%20-%20Incremental%20Build%20-%20QA%20Tests/job/master/5/testReport/junit/api2/dyndns/Run_Tests___API_2_0___test_01_Updating_Settings_for_Custom_Provider/
ISO: https://builds.ixsystems.com/jenkins/job/FreeNAS%20-%20Master%20-%20Incremental%20Build/4181/artifact/artifacts/iso/FreeNAS-11.3-MASTER-201901221710-b20c912f1.iso
Logs: https://builds.ixsystems.com/jenkins/job/FreeNAS%20-%20Incremental%20Build%20-%20QA%20Tests/job/master/5/

here is the error ouput from a local run.

@custom_test_cfg
def test_01_Updating_Settings_for_Custom_Provider():
    global test
    results = PUT('/dyndns/', {
        'username': 'foo',
        'password': 'bar',
        'provider': 'ixsystems.com',
        'domain': ['foobar']})
>       assert results.status_code == 200, results.text
E       AssertionError: {
E          "datastore_update.custom_ddns_server": [
E           {
E            "message": "custom_ddns_server field not recognized",
E            "errno": 22
E           }
E          ],
E          "datastore_update.custom_ddns_path": [
E           {
E            "message": "custom_ddns_path field not recognized",
E            "errno": 22
E           }
E          ]
E         }
E       assert 422 == 200
E        +  where 422 = <Response [422]>.status_code

I have also try to use custom_ddns_server and custom_ddns_path, I get the same error field not recognized.

Associated revisions

Revision a8d79c83 (diff)
Added by Waqar Ahmed almost 2 years ago

Replace prefix appropriately This commit fixes an issue with the datastore service where we replaced all instances of a prefix in the field name. This changes that behaviour to only replace the first occurrence. Ticket: #71635

History

#1 Updated by Dru Lavigne almost 2 years ago

  • Assignee changed from Release Council to William Grzybowski

#2 Updated by William Grzybowski almost 2 years ago

  • Assignee changed from William Grzybowski to Waqar Ahmed
  • Target version changed from Backlog to 11.3-BETA1

#3 Updated by Waqar Ahmed almost 2 years ago

  • Status changed from Unscreened to In Progress

#5 Updated by Waqar Ahmed almost 2 years ago

  • Status changed from In Progress to Ready for Testing

#6 Updated by Eric Turgeon almost 2 years ago

  • Status changed from Ready for Testing to Passed Testing

#7 Updated by Dru Lavigne almost 2 years ago

  • Subject changed from API v2 dyndns return 422 whit custom_ddns_server and custom_ddns_path field not recognized to Replace prefix appropriately for datastore service in API
  • Status changed from Passed Testing to Done
  • Needs QA changed from Yes to No
  • Needs Doc changed from Yes to No
  • Needs Merging changed from Yes to No

#8 Updated by Dru Lavigne almost 2 years ago

  • Target version changed from 11.3-BETA1 to Master - FreeNAS Nightlies

Also available in: Atom PDF