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 os_distro and os_version in OpenStack images #1981 #1982

Merged
merged 4 commits into from
Apr 18, 2024

Conversation

micafer
Copy link
Contributor

@micafer micafer commented Jan 3, 2024

Add os_distro and os_version in OpenStack images

Description

Implements: #1981

Status

Replace this: describe the PR status. Examples:

  • done, ready for review

Checklist (tick everything that applies)

  • Code linting (required, can be done after the PR checks)
  • Documentation
  • Tests
  • ICLA (required for bigger changes)

@codecov-commenter
Copy link

Codecov Report

Merging #1982 (0ee184a) into trunk (6e39117) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Additional details and impacted files
@@           Coverage Diff           @@
##            trunk    #1982   +/-   ##
=======================================
  Coverage   83.20%   83.20%           
=======================================
  Files         353      353           
  Lines       81433    81435    +2     
  Branches     8593     8593           
=======================================
+ Hits        67751    67753    +2     
  Misses      10874    10874           
  Partials     2808     2808           
Files Coverage Δ
libcloud/compute/drivers/openstack.py 84.93% <ø> (ø)
libcloud/test/compute/test_openstack.py 94.54% <100.00%> (+0.01%) ⬆️

Copy link
Member

@Kami Kami left a comment

Choose a reason for hiding this comment

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

LGTM, thanks.

@Kami Kami added this to the v3.9.0 milestone Apr 18, 2024
@Kami Kami merged commit 1eec0cd into apache:trunk Apr 18, 2024
16 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants