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

PSCE-243 #53

Closed
wants to merge 2 commits into from
Closed

PSCE-243 #53

wants to merge 2 commits into from

Conversation

afflom
Copy link
Contributor

@afflom afflom commented Oct 5, 2023

No description provided.


runs:
using: "docker"
image: "Dockerfile"
Copy link
Member

Choose a reason for hiding this comment

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

Is this a path? Would it be different now that the action file is not in the root of the directory?

@afflom afflom changed the title feat(action): prep for multiple actions PSCE-243 Oct 5, 2023
Related: PSCE-243
Signed-off-by: Alex Flom <[email protected]>
Copy link
Member

@jpower432 jpower432 left a comment

Choose a reason for hiding this comment

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

Added some questions here about the overall approach.

trestlebot/infra/entrypoints/create_ssp.py Outdated Show resolved Hide resolved
trestlebot/infra/entrypoints/create_ssp.py Outdated Show resolved Hide resolved
trestlebot/infra/entrypoints/create_ssp.py Outdated Show resolved Hide resolved
actions/create-ssp/action.yml Outdated Show resolved Hide resolved
trestlebot/infra/entrypoints/create_ssp.py Outdated Show resolved Hide resolved
- Consolidates multiple trestle-bot entrypoints within the
published trestle-bot image.

Related PSCE-243, PSCE-250

Signed-off-by: Alex Flom <[email protected]>
@@ -81,7 +81,7 @@ outputs:

runs:
using: "docker"
image: "Dockerfile"
image: "quay.io/continuouscompliance/trestlebot:latest"
Copy link
Member

@jpower432 jpower432 Oct 10, 2023

Choose a reason for hiding this comment

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

If we point to the pre-built image, I would prefer we not point to latest. Pointing to latest means that older versions of the trestlebot action could be pointing to the newest image and breaking changes could make older version of the action unusable (i.e. inputs are expected, but not provided in the action.yml)

Copy link
Member

@jpower432 jpower432 left a comment

Choose a reason for hiding this comment

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

I like this approach. Do you think we could transform the autofix.sh to python? Maybe on another PR? It would be really useful in #55 if I could reuse some of that logic.

@afflom
Copy link
Contributor Author

afflom commented Oct 12, 2023

'twas cherry-picked. Closing

@afflom afflom closed this Oct 12, 2023
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

Successfully merging this pull request may close these issues.

2 participants