Skip to content

Create and upload ssdlc_compliance_report.md #1405

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

Merged
merged 5 commits into from
Jun 3, 2024

Conversation

stIncMale
Copy link
Member

@stIncMale stIncMale commented May 29, 2024

The SSDLC artifacts are created and uploaded to an AWS S3 bucket when we run either publish-snapshot or publish-release Evergreen task. An example of the report is here, it was created by this publish-snapshot run.

JAVA-5435

@stIncMale stIncMale self-assigned this May 29, 2024
@stIncMale stIncMale force-pushed the JAVA-5435 branch 9 times, most recently from 2a5f103 to 068ed0c Compare May 31, 2024 00:38
JAVA-5435
@stIncMale stIncMale marked this pull request as ready for review May 31, 2024 03:30
@stIncMale stIncMale requested a review from jyemin May 31, 2024 03:31
@jyemin jyemin requested a review from rtimmons May 31, 2024 12:37
@jyemin
Copy link
Collaborator

jyemin commented May 31, 2024

@rtimmons if you have the time please give this PR a look.

stIncMale added 2 commits May 31, 2024 09:39
JAVA-5435
JAVA-5435
Copy link
Collaborator

@jyemin jyemin left a comment

Choose a reason for hiding this comment

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

Just one request, otherwise LGTM

@@ -0,0 +1,82 @@
# ${product_name} SSDLC compliance report
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can we hide this file inside the .evergreen directory. It seems too prominent at the top level like this.

Copy link
Member Author

Choose a reason for hiding this comment

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

Done in 06ec2e9.

JAVA-5435
@stIncMale stIncMale requested a review from jyemin May 31, 2024 20:16
Go to
<a href="https://papertrail.devprod-infra.prod.corp.mongodb.com/product-version?product=${product_name}&version=${product_version}">
https://papertrail.devprod-infra.prod.corp.mongodb.com/product-version?product=${product_name}&version=${product_version}</a>,
look at the value in the "Submitter" column.

Choose a reason for hiding this comment

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

@dylrich are we comfortable with this?

Copy link
Collaborator

Choose a reason for hiding this comment

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

If we're going to include these links in the report, maybe we should pull their templates from Evergreen project properties instead of checking them in to the repo.

Copy link
Contributor

@dylrich dylrich May 31, 2024

Choose a reason for hiding this comment

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

The UI has no long-term stability guarantees and is intended purely for debugging. It's not an Official(tm) product we support. We have not received direction on an official way to serve Papertrail data yet. I'd be more comfortable with language that says something more vague like "see Papertrail"

Copy link
Member Author

Choose a reason for hiding this comment

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

Done in 98c5113.

JAVA-5435
@stIncMale stIncMale requested a review from dylrich June 3, 2024 14:32
Copy link
Contributor

@dylrich dylrich left a comment

Choose a reason for hiding this comment

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

LGTM

@stIncMale stIncMale merged commit 339bd2c into mongodb:master Jun 3, 2024
59 checks passed
@stIncMale stIncMale deleted the JAVA-5435 branch June 3, 2024 16:10
jyemin pushed a commit that referenced this pull request Jun 7, 2024
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.

None yet

4 participants