Skip to content

feat: add course export for TC_DEVOPS_16 (cert autogeneration waffle)#451

Merged
kdmccormick merged 3 commits intomainfrom
kdmccormick/tc-devops-16-tar-gz
May 7, 2025
Merged

feat: add course export for TC_DEVOPS_16 (cert autogeneration waffle)#451
kdmccormick merged 3 commits intomainfrom
kdmccormick/tc-devops-16-tar-gz

Conversation

@kdmccormick
Copy link
Copy Markdown
Member

Will link this in the test sheet

@kdmccormick kdmccormick requested review from feanil and sarina April 29, 2025 18:28
@@ -0,0 +1,3 @@
This is a place to put course exports or other helpful resources for executing named release tests.

When adding a file here, please clearly identify the test case(s) for which the file is useful.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Could you add a note about what this course you're adding is useful for? :)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

For this course, I assume the export isn't enough for the test - you also need someone to turn on a waffle flag, yeah?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

The name of the file is tc-devops-16.tar.gz, I figured that would suffice? That way this readme can't fall out of date.

Yes, a waffle flag is needed. I was hoping that the test instructions would stay in the test sheet, and that this would just be an artifact that someone can grab in order to skip some setup steps. i.e.:

SETUP

  • Flip waffle X
  • Create a new course
  • Download and import (link to this tar) OR follow these steps to manually prepare the course using these steps:
    • ... original setup steps

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

gotcha, there's always the risk that the test accidentally gets deleted or renamed but if you think that's not a risk, we can merge

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@sarina I edited the README to hopefully be a little clearer, LMK what you think :)

In theory, the unique test IDs shouldn't change, even if the test is renamed. They're supposed to be stable identifiers that we can compare release to release.

@kdmccormick kdmccormick merged commit c7e5b19 into main May 7, 2025
2 checks passed
@kdmccormick kdmccormick deleted the kdmccormick/tc-devops-16-tar-gz branch May 7, 2025 15:33
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