Skip to content

python-common, spec.in: Add missing pyyaml dependency#36363

Merged
tchaikov merged 1 commit intoceph:masterfrom
sebastian-philipp:p-common-pyyaml
Jul 30, 2020
Merged

python-common, spec.in: Add missing pyyaml dependency#36363
tchaikov merged 1 commit intoceph:masterfrom
sebastian-philipp:p-common-pyyaml

Conversation

@sebastian-philipp
Copy link
Contributor

@sebastian-philipp sebastian-philipp commented Jul 30, 2020

Fixes: cd6a488
Fixes: https://tracker.ceph.com/issues/46759
Signed-off-by: Sebastian Wagner sebastian.wagner@suse.com

Checklist

  • References tracker ticket
  • Updates documentation if necessary
  • Includes tests for new functionality or reproducer for bug
Show available Jenkins commands
  • jenkins retest this please
  • jenkins test classic perf
  • jenkins test crimson perf
  • jenkins test signed
  • jenkins test make check
  • jenkins test make check arm64
  • jenkins test submodules
  • jenkins test dashboard
  • jenkins test dashboard backend
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox

@sebastian-philipp
python-common, spec.in: Add missing pyyaml dependency
0ada965 
Fixes: cd6a488
Fixes: https://tracker.ceph.com/issues/46759
Signed-off-by: Sebastian Wagner <sebastian.wagner@suse.com>
@sebastian-philipp sebastian-philipp requested a review from jan--f July 30, 2020 07:55
jan--f
jan--f approved these changes Jul 30, 2020
View reviewed changes
Copy link
Contributor

@jan--f jan--f left a comment

Choose a reason for hiding this comment

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

lgtm

@tchaikov
Copy link
Contributor

tchaikov commented Jul 30, 2020

@sebastian-philipp could you verify if the debian package of python3-ceph-common 's dependencies are updated as well? in theory ${python3:Depends} should take care of this. just in case..

@sebastian-philipp
Copy link
Contributor Author

sebastian-philipp commented Jul 30, 2020

@sebastian-philipp could you verify if the debian package of python3-ceph-common 's dependencies are updated as well? in theory ${python3:Depends} should take care of this. just in case..

My thought as well

@tchaikov
Copy link
Contributor

tchaikov commented Jul 30, 2020 

Package: python3-ceph-common
Source: ceph
Version: 16.0.0-3985-g61c2edae794-1bionic
Architecture: all
Maintainer: Ceph Maintainers <ceph-maintainers@lists.ceph.com>
Installed-Size: 141
Depends: python3-yaml, python3:any (>= 3.3.2-2~)
Section: python
Priority: optional
Homepage: http://ceph.com/
Description: Python 3 utility libraries for Ceph
 Ceph is a massively scalable, open-source, distributed
 storage system that runs on commodity hardware and delivers object,
 block and file system storage.
 .
 This package contains data structures, classes and functions used by Ceph.
 It also contains utilities used for the cephadm orchestrator.

@sebastian-philipp confirmed.

@tchaikov tchaikov merged commit 70232e1 into ceph:master Jul 30, 2020
@jan--f jan--f mentioned this pull request Aug 11, 2020
Merged
@smithfarm smithfarm mentioned this pull request Aug 14, 2020
Merged
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tchaikov tchaikov tchaikov approved these changes

+1 more reviewer

@jan--f jan--f jan--f approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

bug-fix ceph-volume cephadm common needs-qa wip-kefu-testing

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@sebastian-philipp @tchaikov @jan--f