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

Update the skip condition #11648

Closed
wants to merge 1 commit into from

Conversation

nhe-NV
Copy link
Contributor

@nhe-NV nhe-NV commented Feb 6, 2024

Add some skip condition based on the github issue.

Description of PR

Summary: Add some skip conditions.

  1. skip test_bfd.py::test_bfd_multihop and test_bgp_queue.py::test_bgp_queues due to the limitation on Mellanox platform.
  2. skip test_fgnhg.py on Mellanox SPC2 by [tests/ecmp/inner_hashing] and [tests/ecmp] test failure #6558.
  3. skip generic_config_updater/test_eth_interface.py::test_update_speed by generic_config_updater/test_eth_interface.py failed #8143 and [Config Updater] Rollback failed after port speed is changed sonic-buildimage#13267.
  4. skip platform_tests/test_advanced_reboot.py::test_fast_reboot on Mellanox SPC2 by [systemd][teamsyncd] missing logs during restarting system logging service sonic-buildimage#17792.
  5. skip platform_tests/test_service_warm_restart.py by Test test_service_warm_restart gets stuck in dead loop. #10362.
  6. skip test_nbr_health.py by [test_nbr_health] [test issue] invalid literal for int() with base 10: 'No more variables left in this MIB View #7883

Fixes # (issue)

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Back port request

  • 201911
  • 202012
  • 202205
  • 202305
  • 202311

Approach

What is the motivation for this PR?

How did you do it?

How did you verify/test it?

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

Update the skip condition

Change-Id: I25384eefc8c847d2d33426727e057e1362d4cabf
@nhe-NV
Copy link
Contributor Author

nhe-NV commented Feb 7, 2024

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@liat-grozovik
Copy link
Collaborator

@nhe-NV the PR description is lack of information what tests are now in ignored. can you please provide the info?
also if the issues are due to different reasons or type of tests i think it is better to have different pr per issue for better tracking

@nhe-NV
Copy link
Contributor Author

nhe-NV commented Feb 19, 2024

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@nhe-NV
Copy link
Contributor Author

nhe-NV commented Mar 3, 2024

Splited this PR to 6 PRs due to skip reason are different:
#11839
#11840
#11841
#11842
#11843
#11844

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants