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

Add APEL Record/Message documentation for grid #672

Merged
merged 5 commits into from
Sep 20, 2024

Conversation

gregcorbett
Copy link
Contributor

Add APEL Record/Message documentation for grid accounting:

  • individual job accounting
  • summary accounting
  • sync records

Based on the content from the EGI Wiki - which can be accessed via The Internet Archive: http://web.archive.org/web/20210411151958/https://wiki.egi.eu/wiki/APEL/MessageFormat

I've added this to the internal section of the documentation, as if providers are using "off the shelf" components - they don't need to be overly concerned with the exact format of the record / message.

@gregcorbett
Copy link
Contributor Author

Maybe one for the APEL team to look at as well as part of the review.

@gwarf gwarf added the safe for preview Pull request vetted as safe for preview label Sep 10, 2024
@gwarf
Copy link
Member

gwarf commented Sep 10, 2024

Thanks @gregcorbett, please look at addressing the lint reports.

tofu-rocketry, can you please have a look?

Copy link

Documentation preview deployed!

Available at https://docs.egi.eu/documentation/672

@gwarf
Copy link
Member

gwarf commented Sep 10, 2024

The output of this PR can be checked at https://docs.egi.eu/documentation/672/internal/accounting/record-and-message-formats/

From a quick look, the links on the top page look weird/broken. The link title for "Grid Account record Message formats" may better be shortened, maybe "Grid accounting" could be enough, considering the parent section name.

Copy link

Documentation preview deployed!

Available at https://docs.egi.eu/documentation/672

1 similar comment
Copy link

Documentation preview deployed!

Available at https://docs.egi.eu/documentation/672

- based on the content from the EGI Wiki
Copy link

Documentation preview deployed!

Available at https://docs.egi.eu/documentation/672

@gregcorbett
Copy link
Contributor Author

Lint addressed.

If I'm interpreting the Check links output, the failure isn't due to this PR. It seems to be due to the content/en/providers/cloud-compute/openstack/aai/_index.md page.

@sebastian-luna-valero
Copy link
Member

Yes, thanks!

These were fixed in #673

So merging with the latest main should fix it

Copy link

Documentation preview deployed!

Available at https://docs.egi.eu/documentation/672

Copy link

Documentation preview deployed!

Available at https://docs.egi.eu/documentation/672

Copy link

Documentation preview deployed!

Available at https://docs.egi.eu/documentation/672

Copy link
Member

@gwarf gwarf left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

Documentation preview deployed!

Available at https://docs.egi.eu/documentation/672

@gwarf gwarf merged commit 157ca7a into EGI-Federation:main Sep 20, 2024
8 checks passed
@gregcorbett gregcorbett deleted the apel_grid_v03 branch September 24, 2024 10:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
safe for preview Pull request vetted as safe for preview
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants