Skip to content

[action] [PR:1676] Revert "[meta] do not fail bulk operations if MODE_IGNORE_ERROR (#1613)"#1726

Merged
mssonicbld merged 6 commits intosonic-net:202511from
mssonicbld:cherry/202511/1676
Feb 25, 2026
Merged

[action] [PR:1676] Revert "[meta] do not fail bulk operations if MODE_IGNORE_ERROR (#1613)"#1726
mssonicbld merged 6 commits intosonic-net:202511from
mssonicbld:cherry/202511/1676

Conversation

@mssonicbld
Copy link
Collaborator

This commit caused failures in test_duplicate_route.

This reverts commit fb5652b.

…c-net#1613)"

This commit caused failures in test_duplicate_route.

This reverts commit fb5652b.
@mssonicbld
Copy link
Collaborator Author

Original PR: #1676

@mssonicbld
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

status = meta_generic_validation_get(meta_key, attr_count[idx], attr_list[idx]);

CHECK_STATUS_SUCCESS_MODE(status, mode);
// FIXME: This macro returns on failure.

Check notice

Code scanning / CodeQL

FIXME comment Note

FIXME comment: This macro returns on failure.
@radha-danda
Copy link

@Ndancejic, 1 test failed on this PR - [ FAILED ] FlexCounter.bulkChunksize
Can you please check?

@vmittal-msft - FYI

@yijingyan2
Copy link
Contributor

/azpw run

@mssonicbld
Copy link
Collaborator Author

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld merged commit d9616e9 into sonic-net:202511 Feb 25, 2026
15 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants