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.

Sign up for GitHub

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

Jump to bottom

feat: overhaul CI implementation and use latest pre-salted images #293

Merged
merged 6 commits into from
Mar 7, 2021
Merged

feat: overhaul CI implementation and use latest pre-salted images #293

merged 6 commits into from
Mar 7, 2021

Commits on Mar 7, 2021

  1. feat(openvpn): review PR 134 

    * saltstack-formulas/openvpn-formula#134
    @myii
    myii committed Mar 7, 2021
    Configuration menu
    Copy the full SHA
    ad69201 View commit details
    Browse the repository at this point in the history

  2. fix(template): review PR 212 

    Completes 5b5d679
    @myii
    myii committed Mar 7, 2021
    Configuration menu
    Copy the full SHA
    4c94c9b View commit details
    Browse the repository at this point in the history

  3. feat(java): review PR 15 

    * saltstack-formulas/java-formula#15
    @myii
    myii committed Mar 7, 2021
    Configuration menu
    Copy the full SHA
    92305e3 View commit details
    Browse the repository at this point in the history

  4. feat(java): review PR 17 

    * saltstack-formulas/java-formula#17
    @myii
    myii committed Mar 7, 2021
    Configuration menu
    Copy the full SHA
    c5f9b54 View commit details
    Browse the repository at this point in the history

  5. feat: overhaul CI implementation and use latest pre-salted images 

    Establish all of the following changes:

* Use wildcards for (but *maintain* current ordering of entries when rendered):
  - NOTE: Use new values to ensure reverse compatibility,
          where need to be able to use the original, non-wildcard method instead
  - To filter out `platforms` that must not be included (and `platform_matrix`)
  - `platform_suite` inclusions and exclusions
  - `platforms_matrix` (after initial testing)
* Use latest pre-salted images
  - Active unique combination of platforms for all formulas:
    + E.g. `apt-formula` must only have Debian-based platforms in the list
    + Based around idea of supplying these in the `FORMULA` file
    + Active became the wildcards that have been implemented
* Move `dependencies` to the top in each suite (where used)
* Use new solution to replace `platforms_matrix_commented_includes`:
  - Use wildcards?
  - Automatic generation from "ideal, unique platforms"?

When running global tests across new pre-salted images:

* `vim-interestingwords` is exceptionally useful here, with the following words:
  - `platforms` (use wildcards to ensure all relevant platforms are selected)
  - `supports` (pretty much want to comment all of these out, so InSpec tests run)
  - `include` & `exclude` (similar to `platforms`)
    @myii
    myii committed Mar 7, 2021
    Configuration menu
    Copy the full SHA
    b1753e7 View commit details
    Browse the repository at this point in the history

  6. feat: remove unmanaged formulas (mattermost & stack)

    @myii
    myii committed Mar 7, 2021
    Configuration menu
    Copy the full SHA
    47c3d45 View commit details
    Browse the repository at this point in the history