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

debian packaging question #3

Open
realtime-neil opened this issue Jun 24, 2021 · 7 comments
Open

debian packaging question #3

realtime-neil opened this issue Jun 24, 2021 · 7 comments

Comments

@realtime-neil
Copy link

Is/are the maintainer(s) interested in making this project into a native Debian package? Asking because I use session-manager-plugin on Debian and Ubuntu.

@realtime-neil
Copy link
Author

realtime-neil commented Jun 24, 2021

Follow-up: I know about the "official" package mentioned here...

https://docs.aws.amazon.com/systems-manager/latest/userguide/session-manager-working-with-install-plugin.html#install-plugin-debian

...but there are quite a few policy violations on that one:

$ dpkg-deb --raw-extract session-manager-plugin.deb session-manager-plugin     
$ tree session-manager-plugin
session-manager-plugin
|-- DEBIAN
|   |-- conffiles
|   |-- control
|   |-- postinst
|   |-- postrm
|   |-- preinst
|   `-- prerm
|-- etc
|   `-- init
|       `-- session-manager-plugin.conf
|-- lib
|   `-- systemd
|       `-- system
|           `-- session-manager-plugin.service
`-- usr
    |-- local
    |   `-- sessionmanagerplugin
    |       |-- LICENSE
    |       |-- NOTICE
    |       |-- README.md
    |       |-- RELEASENOTES.md
    |       |-- THIRD-PARTY
    |       |-- VERSION
    |       |-- bin
    |       |   `-- session-manager-plugin
    |       `-- seelog.xml.template
    `-- share
        |-- doc
        |   `-- sessionmanagerplugin
        |       `-- changelog.gz
        `-- lintian
            `-- overrides
                `-- sessionmanagerplugin

15 directories, 18 files

For instance, we don't generally allow packages to install things under /usr/local.

@yuting-fan
Copy link
Contributor

Hi realtime-neil@,

Thank you for reporting the issue with debian paths. Session Manager service teams will evaluate the request and update here.

Thanks,
Yuting

@pwillis-els
Copy link

The file paths are pretty egregious for packaged software. But also the package name ("session-manager-plugin") is so generic that you don't really know what it's for. The package could also use a better description; "see our website for details" isn't a description of software. And it's a plugin for AWS CLI, but it doesn't even list AWS CLI as a package dependency. It also lists docs like LICENSE and README.md as "conffiles".

@realtime-neil
Copy link
Author

With the creation of #30 , I would call this isuse a proper subset of that one.

@tarfeef101
Copy link

Hi realtime-neil@,

Thank you for reporting the issue with debian paths. Session Manager service teams will evaluate the request and update here.

Thanks, Yuting

so... where's that update? can't imagine it's taken almost a year to evaluate?

@realtime-neil realtime-neil reopened this Mar 15, 2022
usk83 pushed a commit to usk83/session-manager-plugin that referenced this issue Nov 28, 2022
@arthurzenika
Copy link

Another question is why does the package require a service session-manager-plugin.service ?

@realtime-neil
Copy link
Author

realtime-neil commented Jun 12, 2023

@arthurzenika

Another question is why does the package require a service session-manager-plugin.service ?

For the same reason everything else is this repository looks like this: Because Amazon.

For lack of any official response (either in the past or likely forthcoming) I'll make a valiant guess: If Amazon can get you to run their program as root on a host you nominally control, then Amazon wants that program running always and forever.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants