Skip to content

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

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
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 <[email protected]>
@stepanblyschak
[sonic-package-manager] build sonic-package-manager and install on SONiC
86a1a44 
Signed-off-by: Stepan Blyshchak <[email protected]>
@stepanblyschak
[sonic-package-manager] if in chroot or database is not running do no…
541ccc0 
…t create DBConnector

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

Signed-off-by: Stepan Blyshchak <[email protected]>
@stepanblyschak
code cleanup
aead06c 
Signed-off-by: Stepan Blyshchak <[email protected]>
@stepanblyschak
allow "manifest" command to accept a reference expression
9aa95d6 
Signed-off-by: Stepan Blyshchak <[email protected]>
@stepanblyschak
remove commented code
8ac0d84 
Signed-off-by: Stepan Blyshchak <[email protected]>
@stepanblyschak
update PackageManager with docstrings
ba3991d 
Signed-off-by: Stepan Blyshchak <[email protected]>
@stepanblyschak
fix s2s migration: return None instead of False
62e23ba 
Signed-off-by: Stepan Blyshchak <[email protected]>
@stepanblyschak
add few migration unit tests
79d9013 
Signed-off-by: Stepan Blyshchak <[email protected]>
@stepanblyschak
[sonic-package-manager] install CLI plugins
cf9d8a9 
Signed-off-by: Stepan Blyshchak <[email protected]>
@stepanblyschak
[sonic-package-manager] support changelog
10a84b1 
Signed-off-by: Stepan Blyshchak <[email protected]>
@stepanblyschak
fix unmarshaling bug
dec2490 
Signed-off-by: Stepan Blyshchak <[email protected]>
@stepanblyschak
fix build-in uninstall, upgrade bug
ebc4845 
Signed-off-by: Stepan Blyshchak <[email protected]>
@stepanblyschak
add debug log for template rendering
31e6383 
Signed-off-by: Stepan Blyshchak <[email protected]>
@stepanblyschak
add missing newlines
7499fdf 
Signed-off-by: Stepan Blyshchak <[email protected]>
@stepanblyschak
add docstrings to registry.py
0fb254d 
Signed-off-by: Stepan Blyshchak <[email protected]>
@stepanblyschak
add docstrings to creator.py
887d730 
Signed-off-by: Stepan Blyshchak <[email protected]>
@stepanblyschak
change dependency from SWSSSDK_PY2 to SWSSSDK_PY3
296127d 
Signed-off-by: Stepan Blyshchak <[email protected]>
@stepanblyschak
put a comment about swsssdk dependency
f3e3ea8 
Signed-off-by: Stepan Blyshchak <[email protected]>
@stepanblyschak
generate a header for monit.conf template
596267b 
Signed-off-by: Stepan Blyshchak <[email protected]>
@stepanblyschak
remove PEER and simplify service_mgmt.sh.j2
7797b99 
Signed-off-by: Stepan Blyshchak <[email protected]>
@stepanblyschak
remove default Before=ntp-config.service
89e0e3f 
Signed-off-by: Stepan Blyshchak <[email protected]>
@stepanblyschak
[database] internal review comments fixes
d731b30 
Signed-off-by: Stepan Blyshchak <[email protected]>
@stepanblyschak
[dockerapi.py] fixes per internal review
aa178e9 
Signed-off-by: Stepan Blyshchak <[email protected]>
@stepanblyschak
[logger] dont pring "info" prefix for info logs
d2dc252 
Signed-off-by: Stepan Blyshchak <[email protected]>
@stepanblyschak
[progress] fixes per internal review
d048d73 
Signed-off-by: Stepan Blyshchak <[email protected]>
@stepanblyschak
[reference] add missing newline at the end of file
2df144d 
Signed-off-by: Stepan Blyshchak <[email protected]>
@stepanblyschak
[registry] fixes per internal review
d21e009 
Signed-off-by: Stepan Blyshchak <[email protected]>
@stepanblyschak
[manager] fix enabled state
1465515 
Signed-off-by: Stepan Blyshchak <[email protected]>
@stepanblyschak
[service_creator] support tmpfs mount
61a24b2 
Signed-off-by: Stepan Blyshchak <[email protected]>
@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 <[email protected]>
@stepanblyschak
[sonic_db] remove swsssdk dependency
7f60f78 
Signed-off-by: Stepan Blyshchak <[email protected]>
@stepanblyschak
[manager] fix review comments
8c69739 
Signed-off-by: Stepan Blyshchak <[email protected]>
@lguohan lguohan added the appext Application Extension label Jan 22, 2021
@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 <[email protected]>
@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 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 <[email protected]>
@stepanblyschak
[manager] implement component version checks
d94577a 
Signed-off-by: Stepan Blyschak <[email protected]>
@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 <[email protected]>
@stepanblyschak
[manager] add option to skip CLI installation
aaa373d 
Signed-off-by: Stepan Blyschak <[email protected]>
@stepanblyschak
[metadata] remove unneded constants
e073593 
Signed-off-by: Stepan Blyschak <[email protected]>
@stepanblyschak
temp
233ab83 
Signed-off-by: Stepan Blyschak <[email protected]>
@stepanblyschak
[manager] base os per component version check
a280970 
Signed-off-by: Stepan Blyschak <[email protected]>
@stepanblyschak
Copy link
Collaborator Author

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

@stepanblyschak stepanblyschak deleted the spm branch September 23, 2022 13:33
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