Singularity: [CHECKS] add more checks!

Created on 22 Jul 2017  路  13Comments  路  Source: hpcng/singularity

Inspiration from these sources

https://hackerone.com/ibb-python
https://internetbugbounty.org/

We would want (generally) tags that check security (for different OS) along with docker specific bugs. We will also eventually have checks for a particular file system structure, but those will come later.

help wanted

Most helpful comment

@vsoch Sure! The best place to initially look will be the INSTALL.md file. It should provide pretty decent instructions on how to get golang installed on your system, as well as where you need to clone the singularity repository. Also it'll walk you through building singularity 3.0 using mconfig

As far as I understand checks, we'll need to add some new commands (src/cmd/singularity/cli) to actually run the checks. Also we'll need to add some stuff to the build command (I think, right??). If so, you'll want to talk with @ikaneshiro on where to interject there. He's driving the image build effort right now.

As far as apps go, I think @oddodaoddo had said he would be investigating porting apps to 3.0. He's trying to start getting into contributing to Singularity, so maybe you two could collaborate together on the Singularity slack channel on getting those features developed? I can provide as much development guidance as needed to help you guys understand the new code tree structure.

Also, sorry that it appears we haven't yet put some of those features you developed back into Singularity 3.0. Our intention was to develop out the core container runtime first and then start making sure that we get everything from the 2.x series implemented again. We're starting to get to the point where we can go back to 2.x and make sure we're hitting everything (check out the test porting PR's that @tri-adam has been working on and merging). We definitely want all of your work in 3.0 :)

All 13 comments

Could anyone point me toward where checks migrated to, if they still exist? The docs say to look at check.sh, but that file was removed after release 2.6.0 and now I can't find where the new source might be.

I was going to submit a PR to fix the docs to point at the 2.6.0 version, but I realized it might still be around somewhere. Thanks.

RIP checks, one of the many contributions I made that is being erased....

no idea, sorry friend.

I asked around a while ago on Slack if anyone was using checks and I just got silence. So, unless there is some interest (maybe you can say there is) I don't think there is a plan to port them over to 3.0.

(Also note that master is 3.0 development. checks still exist in 2.6. Have a look at the 2.6.0 tag.)

@vsoch would you have any interest in helping to port this and other features (apps) over to 3.0?

yes ! :heart:

Awesome! Can use all the help we can get! PR away!

Can you point me to developer docs to get started? I did a PR a long time ago for Singularity Hub, but I still would need to figure out how to develop. I think I had suggested a Docker container but that wasn't liked, so I would rather start with your suggested development environment.

@bauerm97 are you able to point v in the right direction?

@vsoch Sure! The best place to initially look will be the INSTALL.md file. It should provide pretty decent instructions on how to get golang installed on your system, as well as where you need to clone the singularity repository. Also it'll walk you through building singularity 3.0 using mconfig

As far as I understand checks, we'll need to add some new commands (src/cmd/singularity/cli) to actually run the checks. Also we'll need to add some stuff to the build command (I think, right??). If so, you'll want to talk with @ikaneshiro on where to interject there. He's driving the image build effort right now.

As far as apps go, I think @oddodaoddo had said he would be investigating porting apps to 3.0. He's trying to start getting into contributing to Singularity, so maybe you two could collaborate together on the Singularity slack channel on getting those features developed? I can provide as much development guidance as needed to help you guys understand the new code tree structure.

Also, sorry that it appears we haven't yet put some of those features you developed back into Singularity 3.0. Our intention was to develop out the core container runtime first and then start making sure that we get everything from the 2.x series implemented again. We're starting to get to the point where we can go back to 2.x and make sure we're hitting everything (check out the test porting PR's that @tri-adam has been working on and merging). We definitely want all of your work in 3.0 :)

Thanks @bauerm97!

Cool thanks guys! I have definitely missed this github fun banter :) I think a good start would be to create developer docs, proper! Akin to https://github.com/singularityware/singularity-userdocs but something like singularity-devdocs. I would like to do this first, and @bauerm97 I'll use the good resources that you sent along. Does that seem like a good start? Do you like the name ok? --> https://github.com/singularityware/singularity-devdocs

I'd also be glad to chip in, for what it's worth鈥擨 enjoyed working with Golang at a previous job, and I was initially looking for the checks because I wanted to add more.

@rabdill feel free to ping me on the Singularity slack channel (mibauer) and I can help point you (or really anybody that wants to contribute) to the right place.

@vsoch To be honest, I'm not sure we want a singularityware/singularity-devdocs repo right now. Pretty soon we'll be able to start generating a level of developer documentation using go doc, which should be able to guide developers in understanding what different functions and packages are responsible for. At some point, maybe there should just be a top-level CONTRIBUTING.md file in the singularityware/singularity repository which can contain a bit of generic info about different systems, who maintains what, the PR/review process, etc... Maybe once we have a ton of information it will make sense to move into its own repo?

Closing. If a checks infrastructure is a PR contribution to Singularity (perhaps a plugin) obviously this can/should be reopened.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

chrisgorgo picture chrisgorgo  路  3Comments

Amir-Arsalan picture Amir-Arsalan  路  3Comments

zhcf picture zhcf  路  3Comments

kkmann picture kkmann  路  4Comments

jmdf picture jmdf  路  4Comments