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

[YUNIKORN-2373] Extend EventRecord type user/group related data #121

Closed
wants to merge 1 commit into from

Conversation

pbacsko
Copy link
Contributor

@pbacsko pbacsko commented Feb 5, 2024

What is this PR for?

Add new types and constants to the EventRecord type so we can support UGM events.

What type of PR is it?

  • - Bug Fix
  • - Improvement
  • - Feature
  • - Documentation
  • - Hot Fix
  • - Refactoring

Todos

  • - Task

What is the Jira issue?

https://issues.apache.org/jira/browse/YUNIKORN-2373

How should this be tested?

Screenshots (if appropriate)

Questions:

  • - The licenses files need update.
  • - There is breaking changes for older versions.
  • - It needs documentation.

@pbacsko pbacsko self-assigned this Feb 5, 2024
scheduler-interface-spec.md Outdated Show resolved Hide resolved
Comment on lines +931 to +934
UG_USER_LIMITSET = 600; // Limit is configured for a given user
UG_GROUP_LIMITSET = 601; // Limit is configured for a given group
UG_APP_LINK = 602; // Linkage is created between an application and a group
UG_APP_UNLINK = 603; // Unlinking an application and a group
Copy link
Contributor Author

Choose a reason for hiding this comment

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

These were discussed w/ Mani.

@pbacsko pbacsko changed the title [YUNIKORN-2373] Extend EventRecord type with ask and user/group related data [YUNIKORN-2373] Extend EventRecord type user/group related data Feb 5, 2024
Copy link
Contributor

@wilfred-s wilfred-s left a comment

Choose a reason for hiding this comment

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

LGTM, @manirajv06 if you have no further comments we can commit this

@pbacsko pbacsko closed this in 999a361 Feb 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants