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.

Sign up for GitHub

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

Jump to bottom

add resource priority #45

Merged
merged 1 commit into from
Jun 24, 2024
Merged

add resource priority #45

merged 1 commit into from
Jun 24, 2024

Conversation

strokyl
Copy link
Collaborator

@strokyl strokyl commented Jun 24, 2024 •
edited
Loading

This PR introduces the following enhancements to the CLI:

  • It now parses resource priorities from the OpenAPI if they are present.
  • When applying multiple resources simultaneously, they are sorted by priority.

For example, this PR resolves the issue where apply -f fails if an application instance definition precedes its application definition.

@strokyl strokyl force-pushed the add_resource_priority branch 2 times, most recently from 3f22ac9 to 993b91e Compare June 24, 2024 15:21
@strokyl strokyl merged commit c7372a4 into main Jun 24, 2024
4 checks passed
@strokyl strokyl deleted the add_resource_priority branch June 24, 2024 16:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sderosiaux sderosiaux sderosiaux left review comments

@gbecan gbecan gbecan approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@strokyl @gbecan @sderosiaux