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

Known issue: compute environments disappear in CI #606

Closed
calina-c opened this issue Feb 27, 2023 · 2 comments · Fixed by #610
Closed

Known issue: compute environments disappear in CI #606

calina-c opened this issue Feb 27, 2023 · 2 comments · Fixed by #610
Assignees
Labels
Type: Bug Something isn't working

Comments

@calina-c
Copy link
Contributor

calina-c commented Feb 27, 2023

When running the CI, the list of c2d environments disappears for no apparent reason. This affects provider and ocean.py CIs, effectively blocking further development.

We thought it was temporarily fixed by waiting through a retry mechanism here: #603 but it seems that fix is not enough.

@mariacarmina
Copy link
Member

UPDATE:

Checked the connection in this PR on provider https://github.com/oceanprotocol/provider/pull/610/files where I modified the workflow and an old URL and it seems that always the connection is successful https://github.com/oceanprotocol/provider/actions/runs/4345076005/jobs/7589748740#step:8:1, but when provider try to connect to http://172.15.0.13:31000, it occurs Connection refused.

@mariacarmina
Copy link
Member

Approaches for debug shown here

@mariacarmina mariacarmina self-assigned this Mar 14, 2023
mariacarmina added a commit that referenced this issue Apr 13, 2023
* Increased no retries and backoff factor.

* updated workflow.

* tweak.

* tweak 2.

* updated workflow 2.

* Changed config.ini op serv url.

* modified c2d envs.

* Updated config.ini

* refactor tweak.

* print url.

* updated url for c2d envs.

* revert changes. added slash

* tweak

* tweak 22.

* revert

* fix web3 package path.

* added a dict

* more changes

* changed into capital T.

* removed additional dict

* added connection close back.

* check fw.

* Check with updated op serv.

* added prints. modified workflow

* tinny fix.

* eval.

* debug string format.

* tweak 3

* removed eval.

* removed string escape.

* removed urljoin. updated workflow.

* Update pytest.yml

* Update pytest.yml

* added user agent

* fix pytest skip

* add e different version of python.

* add docker ps.

* tweaks to the workflow.

* stop fw.

* add traceroute

* add traceroute host

* chaned op serv url.

* remove stop fw.

* remove traceroute

* added self hosted runner.

* revert code changes from debug process.

* removed docker ps.

* downgrade ubuntu.

* added back self-hosted runner.

* Try with a larger runner.

* try with ubuntu 22 with 4 cores.

* Return to self-hosted env.

* Removed tests for debugging.

* Deleted more tests.

* Removed the graph from CI.

* Added back the tests.

* Test only download.

* Fixed workflow.

* Test only encryption.

* Test only fileinfo.

* Test only graphql tests.

* Deleted graphql tests. UpTest only initialize.

* Added socket test. Added timestamps to each test.

* Decresed number of retries.

* Added back graphql. Modified workflow to skip subgraph deployment.

* black.

* Commented mint_fake_ocean.

* replace with print.

* Test only one test.

* Test the ipfs url test.

* Added logs for ipfs.

* Modified ipfs test.

* Updated ipfs test.

* black.

* Removed prints. Mint datatokens. Removed helper function.

* Run the whole test suite.

* Tested just graphql.

* Added full graph deployment.

* Added tests back.

* Removed pytest.skip.

* Rename script. Modified graphql test.

* Replaced subgraph URL.

* Adapted test with parameterized query. Added aquarius URL back.

---------

Co-authored-by: Călina Cenan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants