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

[sonic-package-manager] first phase implementation of sonic-package-manager #6076

Closed
wants to merge 96 commits into from
Closed

[sonic-package-manager] first phase implementation of sonic-package-manager #6076

wants to merge 96 commits into from

Conversation

stepanblyschak
Copy link
Collaborator

- Why I did it

- How I did it

- How to verify it

- Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

@stepanblyschak
[sonic-package-manager] first phase implementation of sonic-package-m…
2e01350 
…anager

Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
[sonic-package-manager] build sonic-package-manager and install on SONiC
86a1a44 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
[sonic-package-manager] if in chroot or database is not running do no…
541ccc0 
…t create DBConnector

Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
fix s2s upgrade
47bb9bf 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
fix problem that list required root
25c0e8f 
add SonicDB class for creating DB tables.

Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
code cleanup
aead06c 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
allow "manifest" command to accept a reference expression
9aa95d6 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
remove commented code
8ac0d84 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
update PackageManager with docstrings
ba3991d 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
fix s2s migration: return None instead of False
62e23ba 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
add few migration unit tests
79d9013 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
[sonic-package-manager] install CLI plugins
cf9d8a9 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
[sonic-package-manager] support changelog
10a84b1 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
fix unmarshaling bug
dec2490 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
fix build-in uninstall, upgrade bug
ebc4845 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
add debug log for template rendering
31e6383 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
add missing newlines
7499fdf 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
add docstrings to registry.py
0fb254d 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
add docstrings to creator.py
887d730 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
change dependency from SWSSSDK_PY2 to SWSSSDK_PY3
296127d 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
put a comment about swsssdk dependency
f3e3ea8 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
generate a header for monit.conf template
596267b 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
remove PEER and simplify service_mgmt.sh.j2
7797b99 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
remove default Before=ntp-config.service
89e0e3f 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
[database] internal review comments fixes
d731b30 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
[dockerapi.py] fixes per internal review
aa178e9 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
[logger] dont pring "info" prefix for info logs
d2dc252 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
[progress] fixes per internal review
d048d73 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
[reference] add missing newline at the end of file
2df144d 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
[registry] fixes per internal review
d21e009 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
[manager] fix enabled state
1465515 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
[service_creator] support tmpfs mount
61a24b2 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
[manifest] enhance manifest marshaling code.
c3a5c38 
Define one more abstraction for marshaling/unmarshaling values.
This avoids enumerating built-in types in unmarshal.

Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
[sonic_db] remove swsssdk dependency
7f60f78 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
[manager] fix review comments
8c69739 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@lguohan lguohan added the appext Application Extension label Jan 22, 2021
@stepanblyschak
[dump] support multi-asic container techsupport dump
8da9346 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
add support for installing package by specifying repository and tag
e00f03b 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
take description from package manifest
0235fe4 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
[tests/test_manager] assert that rollback is actually executed
72e9c68 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
[tests/test_manager] assert that rollback is actually executed
d82e0ac 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
use docker_image_id variable instead of docker_image_name
759fbd9 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@lgtm-com
Copy link

lgtm-com bot commented Jan 27, 2021

This pull request introduces 7 alerts when merging 759fbd9 into 9ff5644 - view on LGTM.com

new alerts:

  • 4 for 'import *' may pollute namespace
  • 3 for 'super' in old style class

@stepanblyschak
fix LGTM warningns
0427468 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@lgtm-com
Copy link

lgtm-com bot commented Jan 28, 2021

This pull request introduces 3 alerts when merging 0427468 into 98298f7 - view on LGTM.com

new alerts:

  • 3 for 'super' in old style class

@stepanblyschak
fix migration with new api
282d771 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
@stepanblyschak
inherit PackageSource from object to fix LGTM warn
7f66dcf 
Signed-off-by: Stepan Blyshchak <stepanb@nvidia.com>
stepanblyschak and others added 2 commits February 8, 2021 16:46
@stepanblyschak
fix bug related to dump generation and fix repository not present if
889214b 
installation was from local file or repository

Signed-off-by: Stepan Blyschak <stepanb@nvidia.com>
@stepanblyschak
[manager] implement component version checks
d94577a 
Signed-off-by: Stepan Blyschak <stepanb@nvidia.com>
@lgtm-com
Copy link

lgtm-com bot commented Mar 19, 2021

This pull request introduces 4 alerts when merging d94577a into 28cb43c - view on LGTM.com

new alerts:

  • 3 for Unused import
  • 1 for Variable defined multiple times

@stepanblyschak
fix LGTM alerts
ca969b7 
Signed-off-by: Stepan Blyschak <stepanb@nvidia.com>
@stepanblyschak
[manager] add option to skip CLI installation
aaa373d 
Signed-off-by: Stepan Blyschak <stepanb@nvidia.com>
@stepanblyschak
[metadata] remove unneded constants
e073593 
Signed-off-by: Stepan Blyschak <stepanb@nvidia.com>
@stepanblyschak
temp
233ab83 
Signed-off-by: Stepan Blyschak <stepanb@nvidia.com>
@stepanblyschak
[manager] base os per component version check
a280970 
Signed-off-by: Stepan Blyschak <stepanb@nvidia.com>
@stepanblyschak
Copy link
Collaborator Author

Closing in favour of sonic-net/sonic-utilities#1527

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@renukamanavalan renukamanavalan Awaiting requested review from renukamanavalan

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

@qiluo-msft qiluo-msft Awaiting requested review from qiluo-msft qiluo-msft is a code owner

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

Assignees
No one assigned
Labels
appext Application Extension
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@stepanblyschak @lguohan