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

[v0.19.1]: Temporarily work around deploy build command issue #2292

Merged
merged 4 commits into from
Feb 4, 2025

Conversation

8ball030
Copy link
Collaborator

Proposed changes

Resolves issue with path directory when using default build commands.

Copy link
Collaborator

@Adamantios Adamantios left a comment

Choose a reason for hiding this comment

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

Could you please share more information on how to reproduce this issue?

It worries me a bit that the parent path did not exist. I would like to ensure that the solution does not cover a bug.

@8ball030
Copy link
Collaborator Author

8ball030 commented Jan 27, 2025

In order to reproduce;

A) create a new service (Note, this is to show the bug and can be done in any other repo with a service);

[I] eight@gefion ~/P/S/r/experimentation_station (main)> adev create author/cool_agent
16:52:31 INFO     Creating agent author/cool_agent:latest from template eightballer/frontend_agent                                                                                                   
16:52:40 INFO     Agent published successfully.                                                                                                                                                      
         INFO     Agent cleaned up successfully.                                                                                                                                                     
         INFO     Agent author/cool_agent:latest created successfully 🎉🎉🎉🎉                                                                                                                       
[I] eight@gefion ~/P/S/r/experimentation_station (main)> adev convert agent-to-service author/cool_agent author/cool_agent --force
16:52:48 INFO     Converting agent author/cool_agent:latest to service author/cool_agent:latest.                                                                                                     
         INFO     Conversion complete. Service is ready! 🚀  

B) try to deploy that service;

autonomy generate-key ethereum -n 1
autonomy fetch --service --local author/cool_agent
cd cool_agent
autonomy deploy build ../keys.json --n 1

image

@Adamantios
Copy link
Collaborator

An update regarding your PR:

  • It seems that there is indeed a bug, which was introduced in v0.18.4.
  • The bug went unnoticed because the Ubuntu unit test job has been timing out while still ❔showing a green status❔.
  • The solution suggested in this PR does not resolve the real source of the issue and will not be necessary after addressing it.

Please bear with us while we investigate the above situation.

@Adamantios Adamantios added the bug Something isn't working label Feb 3, 2025
@8ball030
Copy link
Collaborator Author

8ball030 commented Feb 3, 2025

coolio any idea when this will be resolved, is blocking us haha

That timing out out issue seems quite nasty to be honest.

You can specifically see its a cancelled job as well.

@Adamantios Adamantios changed the title Update base.py Temporarily work around deploy build command issue Feb 4, 2025
@Adamantios
Copy link
Collaborator

coolio any idea when this will be resolved, is blocking us haha

Releasing this to unblock you. Will address the issues in #2292 (comment) in the next release.

@Adamantios Adamantios changed the title Temporarily work around deploy build command issue [v0.19.1]: Temporarily work around deploy build command issue Feb 4, 2025
@Adamantios Adamantios merged commit 5fb2c83 into valory-xyz:main Feb 4, 2025
23 checks passed
@8ball030
Copy link
Collaborator Author

8ball030 commented Feb 4, 2025

Thank you alot ser

Adamantios added a commit that referenced this pull request Feb 12, 2025
The deployment build process requires the directories.

This commit properly fixes the issue that was worked around in #2292 and rolled back in 5e41dd7b0582933c16ec2a68ffbc35100e34d0d4.
Adamantios added a commit that referenced this pull request Feb 12, 2025
The deployment build process requires the directories.

This commit properly fixes the issue that was worked around in #2292 and rolled back in e214049.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants