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

Update CI/CD to fix PyPi deployment failure and improve documentation deployment #58

Closed
5 tasks done
jordanpadams opened this issue Apr 24, 2020 · 7 comments
Closed
5 tasks done
Assignees
Labels
bug Something isn't working

Comments

@jordanpadams
Copy link
Member

jordanpadams commented Apr 24, 2020

Tasks

@jordanpadams
Copy link
Member Author

@nutjob4life note the added task above for Improved Regression Test Suite

@nutjob4life
Copy link
Member

@jordanpadams I would like that too. When time pressure isn't an issue, the way I like to work is:

  1. Appraise the latest bug report.
  2. Branch and run the test suite. They should all work.
  3. Add a test case that exposes the new bug.
  4. Run the test suite; it should fail, if not, fix the new test case.
  5. Fix the bug.
  6. Run the suite again; it should succeed; if not, back to step 5.
  7. Commit & PR.

This way, you get a monotonically increasing suite.

Downloading on demand is nice but can present intermittent test failures, esp. in automated nightly test environments. If the bundles aren't too large, I sure don't mind including them in the repository. We can omit them from source distributions with the caveat to end-users that "test suite will run in degraded mode" or similar.

What do you think @jordanpadams?

@jordanpadams
Copy link
Member Author

@nutjob4life i will leave it to you. I am definitely OK with this approach, I just know it can be time consuming. so whatever you think is best. at minimum, I would like us to have at least one regression test that runs on a bundle, outputs the AIP and SIP, and do some simple comparisons to ensure the outputs meet some success critieria (e.g. number of lines in .TABs, LIDs, some other maybe useful info)

@nutjob4life
Copy link
Member

Hey @jordanpadams I am completely out of steam since resolving #71 for the PDS.20 milestone.

However the PR for it #72 contains much of the work for the last checkbox on this issue above, Improved Regression Suite. That PR includes:

  • Creating a SIP with all collections = true from the LADEE bundle
  • Creating an AIP with all collections = true from the LADEE bundle
  • Creating a SIP with all collections = true from the Insight Documents bundle
  • Creating an AIP with all collections = true from the Insight Documents bundle
  • Creating a SIP with all collections = false from the Insight Documents bundle
  • Creating an AIP with all collections = false from the Insight Documents bundle

There's a easy-to-plug-in framework in there now to add additional bundles with various parameters.

For this issue, then, that just leaves tagging dev releases which @tloubrieu-jpl may already have implemented. I need to take a closer look at the various GitHub workflows he's added to this repository.

And then deploying dev docs.

Upshot is, I've missed this issue for this milestone. I'm sorry 🙇

@jordanpadams
Copy link
Member Author

@nutjob4life no problem. thanks for getting to #71 ! that is totally OK for this build. this issue is not time sensitive. the other one was so we can get moving on operational release. @tloubrieu-jpl is working on deploying dev vs operational docs so he can probably chime in here and may even wrap up this issue for you.

@tloubrieu-jpl
Copy link
Member

tloubrieu-jpl commented Jul 15, 2020

Update CI/CD to deploy in-dev is difficult to implement because there is not action to push into a specific directory in gh-pages.

It will requires more work and discussion on how to handle automated documentation for software. This will be done in ticket #74

@nutjob4life
Copy link
Member

Note to self @nutjob4life: "changing your point of view" might be a good way to make headway on this effort, similar to how the "corral" does it. For a common approach that can span repositories, we may want to investigate making our own GitHub Action that can be plugged into multiple workflows without mushrooming each to 100s of lines.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants