-
Notifications
You must be signed in to change notification settings - Fork 48
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
Conversation
Maybe one for the APEL team to look at as well as part of the review. |
Thanks @gregcorbett, please look at addressing the lint reports. tofu-rocketry, can you please have a look? |
Documentation preview deployed!Available at https://docs.egi.eu/documentation/672 |
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. |
Documentation preview deployed!Available at https://docs.egi.eu/documentation/672 |
1 similar comment
Documentation preview deployed!Available at https://docs.egi.eu/documentation/672 |
- based on the content from the EGI Wiki
6dca689
to
588f59e
Compare
Documentation preview deployed!Available at https://docs.egi.eu/documentation/672 |
Lint addressed. If I'm interpreting the |
Yes, thanks! These were fixed in #673 So merging with the latest |
Documentation preview deployed!Available at https://docs.egi.eu/documentation/672 |
content/en/internal/accounting/record-and-message-formats/grid-accounting/_index.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Baptiste Grenier <[email protected]>
Documentation preview deployed!Available at https://docs.egi.eu/documentation/672 |
Documentation preview deployed!Available at https://docs.egi.eu/documentation/672 |
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.
LGTM
Documentation preview deployed!Available at https://docs.egi.eu/documentation/672 |
Add APEL Record/Message documentation for grid accounting:
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.