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

Fixing vm tests #3826

Merged
merged 3 commits into from
Jan 30, 2025
Merged

Fixing vm tests #3826

merged 3 commits into from
Jan 30, 2025

Conversation

maayarosama
Copy link
Contributor

@maayarosama maayarosama commented Jan 19, 2025

Description

Fixing vm tests

Changes

  • ● TC1230 - VM: Deploy Multiple VMs on Different Nodes:
    Added a command to install util-linux package
  • ● TC1229 - VM: Deploy a VM With a Disk
    Disk added is in the fifth index not the fourth one

Related Issues

Tested Scenarios

  • Run the tests

To consider

Preliminary Checks:

  • Preliminary Checks
    • Does it completely address the issue linked?
    • What about edge cases?
    • Does it meet the specified acceptance criteria?
    • Are there any unintended side effects?
    • Does the PR adhere to the team's coding conventions, style guides, and best practices?
    • Does it integrate well with existing features?
    • Does it impact the overall performance of the application?
    • Are there any bottlenecks or slowdowns?
    • Has it been optimized for efficiency?
    • Has it been adequately tested with unit, integration, and end-to-end tests?
    • Are there any known defects or issues?
    • Is the code well-documented?
    • Are changes to documentation reflected in the code?

UI Checks:

  • UI Checks
    • If a UI design is provided/ does it follow it?
    • Does every button work?
    • Is the data displayed logical? Is it what you expected?
    • Does every validation work?
    • Does this interface feel intuitive?
    • What about slow network? Offline?
    • What if the gridproxy/graphql/chain is failing?
    • What would a first time user have a hard time navigating here?

Code Quality Checks:

  • Code Quality Checks
    • Code formatted/linted? Are there unused imports? .. etc
    • Is there redundant/repeated code?
    • Are there conditionals that are always true or always false?
    • Can we write this more concisely?
    • Can we reuse this code? If yes, where?
    • Will the changes be easy to maintain and update in the future?
    • Can this code become too complex to understand for other devs?
    • Can this code cause future integration problems?

Testing Checklist

  • Does it Meet the specified acceptance criteria.
  • Test if changes can affect any other functionality.
  • Tested with unit, integration, and end-to-end tests.
  • Tested the un-happy path and rollback scenarios.
  • Documentation updated to meet the latest changes.
  • Check automated tests working successfully with the changes.
  • Can be covered by automated tests.

General Checklist

  • Tests included
  • Build pass
  • Documentation
  • Code format and docstring
  • Screenshots/Video attached (needed for UI changes)

Copy link
Contributor

@amiraabouhadid amiraabouhadid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

@maayarosama
Copy link
Contributor Author

image

Please use your id_ed25519 key not id_rsa key in the config file

Copy link
Contributor

@0oM4R 0oM4R left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

in workflow it fails as well
image

check full run report https://github.com/threefoldtech/tfgrid-sdk-ts/runs/35932577594

@samaradel
Copy link
Contributor

  • ✕ TC2847 - VM: Deploy a VM With Mycelium
    connect ENETUNREACH 4b2:36d6:2576:3e77:ff0f:0:200:2:22 - Local (:::0)
    image

@maayarosama
Copy link
Contributor Author

  • ✕ TC2847 - VM: Deploy a VM With Mycelium
    connect ENETUNREACH 4b2:36d6:2576:3e77:ff0f:0:200:2:22 - Local (:::0)
    image

Mycelium is unstable and once it's fixed the test will pass, so there's nothing to do to fix that test from our side

@maayarosama maayarosama requested a review from 0oM4R January 23, 2025 08:15
@amiraabouhadid
Copy link
Contributor

amiraabouhadid commented Jan 26, 2025

still fails with ed key , deployments/contracts are created and deleted but tests fail
I got this error the first time
image
and this the second time, even tho account is verified
image
might be yggdrassil issue, will test again later

@maayarosama maayarosama merged commit 8c9a1fd into development Jan 30, 2025
9 checks passed
@maayarosama maayarosama deleted the development_vm_tests branch January 30, 2025 15:44
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.

5 participants