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 liquid nova report usage endpoint #611

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Add liquid nova report usage endpoint #611

wants to merge 3 commits into from
+456 −10

Conversation

Varsius
Copy link
Contributor

@Varsius Varsius commented Nov 25, 2024

Checklist:

  • If this PR is about a plugin, I tested the plugin against an OpenStack cluster.
  • I updated the documentation to describe the semantical or interface changes I introduced.

@coveralls
Copy link

coveralls commented Nov 25, 2024 •
edited
Loading

Coverage Status

coverage: 79.312%. remained the same
when pulling a242be7 on liquid-nova-report-usage
into a952441 on master.

majewsky
majewsky reviewed Nov 25, 2024
View reviewed changes
internal/liquids/nova/liquid.go Show resolved Hide resolved
internal/liquids/nova/liquid.go Outdated Show resolved Hide resolved
internal/liquids/nova/liquid.go Outdated Show resolved Hide resolved
@Varsius Varsius force-pushed the liquid-nova-report-usage branch from ce060c8 to 1a17672 Compare November 26, 2024 13:19
Base automatically changed from liquid-nova to master December 17, 2024 12:40
@Varsius Varsius force-pushed the liquid-nova-report-usage branch from d26345b to 121fe20 Compare December 17, 2024 12:53
@Varsius Varsius force-pushed the liquid-nova-report-usage branch from 121fe20 to 9d47f14 Compare December 17, 2024 13:03
@Varsius Varsius marked this pull request as ready for review December 17, 2024 13:04
@Varsius Varsius force-pushed the liquid-nova-report-usage branch from 596f9bd to a242be7 Compare December 17, 2024 13:40
@Varsius Varsius changed the title WIP: Add liquid nova report usage endpoint Add liquid nova report usage endpoint Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@majewsky majewsky majewsky left review comments

@SuperSandro2000 SuperSandro2000 Awaiting requested review from SuperSandro2000 SuperSandro2000 is a code owner

@VoigtS VoigtS Awaiting requested review from VoigtS VoigtS is a code owner

@Nuckal777 Nuckal777 Awaiting requested review from Nuckal777 Nuckal777 is a code owner

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
@Varsius @coveralls @majewsky