-
Notifications
You must be signed in to change notification settings - Fork 2
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
Feature: Job metrics #2036
base: master
Are you sure you want to change the base?
Feature: Job metrics #2036
Conversation
88535eb
to
4e37772
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #2036 +/- ##
==========================================
+ Coverage 49.80% 49.92% +0.11%
==========================================
Files 72 73 +1
Lines 17569 17734 +165
Branches 3418 3437 +19
==========================================
+ Hits 8751 8853 +102
- Misses 8007 8067 +60
- Partials 811 814 +3
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
cd0271d
to
a735d80
Compare
ce0cb6c
to
c933de6
Compare
autosubmit/job/metrics_processor.py
Outdated
with open(path, "r") as f: | ||
content = f.read() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dbeltrankyl is there any defined method to read the file's content on the remote?
I saw the paramiko_platform.py
and there are just methods to move files from remote to local. Still, I'm confused about how to use the Platform class children correctly 😵💫
cc @kinow
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think not. Talking by memory, I believe Autosubmit doesn't read anything from the remote; it only checks some files ( completed, failed wrapper... ) but doesn't read the content
5f16ddb
to
d624b8b
Compare
d624b8b
to
e57278d
Compare
PR for #2031
Example of usage on: #2031 (comment)
Process: