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

[Accounting] Fix issues that were breaking the rendering of accounting info #340

Merged
merged 2 commits into from
Jul 16, 2024

Conversation

gmarciani
Copy link
Collaborator

@gmarciani gmarciani commented Jul 15, 2024

Description

Changes

Fix issues that were breaking the rendering of accounting info
In particular:

  1. fix the retrieval of compute nodes instance type to determine price estimate;
  2. fix the generation of job properties rendered in the job accounting modal.

Also minor improvements to log lines related to accounting.

Screenshot 2024-07-15 at 16 01 42

How Has This Been Tested?

Manually deployed and verified that the modal showed the accounting info per job is displayed.

In order to increase the likelihood of your contribution being accepted, please make sure you have read both the Contributing Guidelines and the Project Guidelines

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

…g info.

 In particular:
 1. fix the retrieval of compute nodes instance type to determine price estimate;
 2. fix the generation of job properties rendered in the job accounting modal.
@gmarciani gmarciani force-pushed the wip/mgiacomo/2024070/fix-acct-0711-1 branch from b8a4830 to cba20d8 Compare July 15, 2024 14:36
@gmarciani gmarciani merged commit c77b415 into aws:main Jul 16, 2024
2 checks passed
@gmarciani gmarciani deleted the wip/mgiacomo/2024070/fix-acct-0711-1 branch July 16, 2024 08:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants