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

🐞 [Bug]: Can't use add_worker in the same script as deploy k8s with Mycelium option is added #3509

Open
1 task done
A-Harby opened this issue Oct 7, 2024 · 9 comments · Fixed by #3512
Open
1 task done
Assignees
Labels
grid_client type_bug Something isn't working
Milestone

Comments

@A-Harby
Copy link
Contributor

A-Harby commented Oct 7, 2024

Is there an existing issue for this?

  • I have searched the existing issues

which package/s did you face the problem with?

grid_client

What happened?

I can deploy a K8s master and worker with Mycelium, but it will fail if I add a worker in the same script with the add_worker function and then run them together.

Steps To Reproduce

If you want to try, you can use the k8s test file; just change all the Mycelium options to turn true and run it; it will fail.

which network/s did you face the problem on?

Dev

version

.

Twin ID/s

No response

Node ID/s

No response

Farm ID/s

No response

Contract ID/s

No response

Relevant screenshots/screen records

image

Relevant log output

ValidationError: Another mycelium seed is used for this network vh71o5bpbu9q34g on this 167
@A-Harby A-Harby added type_bug Something isn't working grid_client labels Oct 7, 2024
@ramezsaeed ramezsaeed added this to 3.15.x Oct 8, 2024
@ramezsaeed ramezsaeed added this to the 2.6.0 milestone Oct 8, 2024
@maayarosama maayarosama removed this from 3.15.x Oct 8, 2024
@maayarosama maayarosama modified the milestones: 2.6.0, 2.7.0 Oct 8, 2024
@github-project-automation github-project-automation bot moved this to Done in 3.16.x Oct 17, 2024
@samaradel samaradel reopened this Oct 24, 2024
@samaradel samaradel moved this from Done to Accepted in 3.16.x Oct 24, 2024
@samaradel samaradel self-assigned this Oct 29, 2024
@samaradel samaradel moved this from Accepted to In Progress in 3.16.x Oct 29, 2024
@samaradel
Copy link
Contributor

Work in Progress (WIP):
Checking myceliumSeeds and how script works

@samaradel samaradel moved this from In Progress to Pending Review in 3.16.x Oct 30, 2024
@samaradel
Copy link
Contributor

Work Completed:

  • Check if myceliumNetworkSeed.seed is existed.
  • Remove .skip from add worker test case.

@atefnazmy
Copy link

  • A similar issue happens when adding a worker via the Dashboard

add_worker

@samaradel samaradel moved this from Pending Review to In Progress in 3.16.x Nov 25, 2024
@samaradel samaradel moved this from In Progress to Blocked in 3.16.x Nov 25, 2024
@samaradel
Copy link
Contributor

Blocked on 3680 as now I got the same error
image

@0oM4R
Copy link
Contributor

0oM4R commented Dec 1, 2024

image
faced this issue again on dashboard

@samaradel samaradel moved this from Blocked to Pending Review in 3.16.x Dec 2, 2024
@AhmedHanafy725 AhmedHanafy725 moved this from Pending Review to In Verification in 3.16.x Dec 18, 2024
@khaledyoussef24
Copy link
Contributor

the deployment works fine but the test keeps failing
may be we shall check the validation or assertions
Screenshot 2024-12-30 at 8 50 01 AM
Screenshot 2024-12-30 at 8 50 14 AM

@khaledyoussef24 khaledyoussef24 moved this from In Verification to Accepted in 3.16.x Dec 30, 2024
@samaradel samaradel moved this from Accepted to In Progress in 3.16.x Jan 30, 2025
@samaradel
Copy link
Contributor

samaradel commented Jan 30, 2025

blocked on #3847 as I can't reproduce the error

@samaradel samaradel moved this from In Progress to Blocked in 3.16.x Jan 30, 2025
@AhmedHanafy725 AhmedHanafy725 moved this from Blocked to In Verification in 3.16.x Feb 2, 2025
@AhmedHanafy725
Copy link
Contributor

blocked on #3847 as I can't reproduce the error

it passes the deployment but it fails in the validation, so it's not the same issue

@khaledyoussef24
Copy link
Contributor

khaledyoussef24 commented Feb 11, 2025

verified and fixed in the workflow
now the test assertion is working fine and the tests are passing fine.

Image and also verified on the dahsboard could deploy vm and add working both uses mycelium Image Image

testcase

@khaledyoussef24 khaledyoussef24 moved this from In Verification to Done in 3.16.x Feb 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
grid_client type_bug Something isn't working
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

8 participants