Skip to content
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

Fix orch stuch when removing vlan member (#3294) #3295

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Merge branch 'master' into orch-stuck-when-remove-vlan-member

611a78b
Select commit
Loading
Failed to load commit list.
Open

Fix orch stuch when removing vlan member (#3294) #3295

Merge branch 'master' into orch-stuck-when-remove-vlan-member
611a78b
Select commit
Loading
Failed to load commit list.
Azure Pipelines / Azure.sonic-swss succeeded Sep 25, 2024 in 2h 26m 16s

Build #20240925.6 had test failures

Details

Tests

  • Failed: 1 (0.06%)
  • Passed: 1,644 (94.00%)
  • Other: 104 (5.95%)
  • Total: 1,749
Code coverage

  • 41073 of 61003 lines covered (67.33%)

Annotations

Check failure on line 7275 in Build log

See this annotation in the file changed.

@azure-pipelines azure-pipelines / Azure.sonic-swss

Build log #L7275

Bash exited with code '123'.

Check failure on line 1 in test_duplicate_eni_route_group

See this annotation in the file changed.

@azure-pipelines azure-pipelines / Azure.sonic-swss

test_duplicate_eni_route_group

AssertionError: Field SAI_ENI_ATTR_OUTBOUND_ROUTING_GROUP_ID in ASIC_DB table ASIC_STATE:SAI_OBJECT_TYPE_ENI not equal to oid:0x14008000000617
Raw output
dash_db = <dash_db.DashDB object at 0x7f67fcdb7ac0>

    def test_duplicate_eni_route_group(dash_db: DashDB):
        dash_db.set_app_db_entry(APP_DASH_ROUTE_GROUP_TABLE_NAME, ROUTE_GROUP1, ROUTE_GROUP1_CONFIG)
        dash_db.set_app_db_entry(APP_DASH_ROUTE_TABLE_NAME, ROUTE_GROUP1, OUTBOUND_ROUTE_PREFIX1, ROUTE_VNET_CONFIG)
        rg1_oid = dash_db.wait_for_asic_db_keys("ASIC_STATE:SAI_OBJECT_TYPE_OUTBOUND_ROUTING_GROUP")[0]
    
        dash_db.set_app_db_entry(APP_DASH_ENI_ROUTE_TABLE_NAME, ENI_ID, ENI_ROUTE_GROUP1_CONFIG)
    
        eni_key = dash_db.wait_for_asic_db_keys("ASIC_STATE:SAI_OBJECT_TYPE_ENI")[0]
>       dash_db.wait_for_asic_db_field("ASIC_STATE:SAI_OBJECT_TYPE_ENI", eni_key, SAI_ENI_ATTR_OUTBOUND_ROUTING_GROUP_ID, rg1_oid)

dash/test_dash_route_group.py:65: 
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 
dash/dash_db.py:148: in wait_for_asic_db_field
    success, value = wait_for_result(polling_function, failure_message=failure_message)
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 

polling_function = <function DashDB.wait_for_asic_db_field.<locals>.polling_function at 0x7f67fcd504c0>
polling_config = PollingConfig(polling_interval=0.01, timeout=20.0, strict=True)
failure_message = 'Field SAI_ENI_ATTR_OUTBOUND_ROUTING_GROUP_ID in ASIC_DB table ASIC_STATE:SAI_OBJECT_TYPE_ENI not equal to oid:0x14008000000617'

    def wait_for_result(
        polling_function: Callable[[], Tuple[bool, Any]],
        polling_config: PollingConfig = PollingConfig(),
        failure_message: str = None,
    ) -> Tuple[bool, Any]:
        """Run `polling_function` periodically using the specified `polling_config`.
    
        Args:
            polling_function: The function being polled. The function cannot take any arguments and
                must return a status which indicates if the function was succesful or not, as well as
                some return value.
            polling_config: The parameters to use to poll the polling function.
            failure_message: The message to print if the call times out. This will only take effect
                if the PollingConfig is set to strict.
    
        Returns:
            If the polling function succeeds, then this method will return True and the output of the
            polling function.
    
            If it does not succeed within the provided timeout, it will return False and whatever the
            output of the polling function was on the final attempt.
        """
        for _ in range(polling_config.iterations()):
            status, result = polling_function()
    
            if status:
                return (True, result)
    
            time.sleep(polling_config.polling_interval)
    
        if polling_config.strict:
            message = failure_message or f"Operation timed out after {polling_config.timeout} seconds with result {result}"
>           assert False, message
E           AssertionError: Field SAI_ENI_ATTR_OUTBOUND_ROUTING_GROUP_ID in ASIC_DB table ASIC_STATE:SAI_OBJECT_TYPE_ENI not equal to oid:0x14008000000617

dvslib/dvs_common.py:60: AssertionError