Skip to content

Re-enable Output Generation CI/CD on PRs - #543

Merged
alecbcs merged 18 commits into
mainfrom
ci/isc26
Jun 19, 2026
Merged

Re-enable Output Generation CI/CD on PRs#543
alecbcs merged 18 commits into
mainfrom
ci/isc26

Conversation

@alecbcs

@alecbcs alecbcs commented Jun 18, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Note: cutting down the develop cache only works because when changing the image we changed the requirements to require x86_64_v2 (from x86_64_v3) which triggered a full rebuild.

Waiting on the release branch to be cut sometime soon by @kwryankrattiger. Will update this PR to use that instead once the cache is populated.

alalazo
alalazo reviewed Jun 18, 2026
View reviewed changes
Comment thread container/Dockerfile Outdated
This was referenced Jun 19, 2026
Merged
Closed
alalazo
alalazo reviewed Jun 19, 2026
View reviewed changes
Comment thread container/packages.yaml Outdated
alecbcs and others added 15 commits June 19, 2026 09:59
@alecbcs
Temporarily disable pushing the changes output to the PR branch
f75bc75 
Signed-off-by: Alec Scott <alec@llnl.gov>
@alecbcs
Bump container to use Ubuntu 26.04
d2d313c 
Signed-off-by: Alec Scott <alec@llnl.gov>
@alecbcs
Cut the develop cache down from 8GB to 3GB
478a53b 
Signed-off-by: Alec Scott <alec@llnl.gov>
@alecbcs
Enforce Spack owns /home/spack
1d3390c 
Signed-off-by: Alec Scott <alec@llnl.gov>
@alecbcs
Allow all users to read/write to container cached home
c65df37 
Signed-off-by: Alec Scott <alec@llnl.gov>
@alecbcs
Swap to releases/v1.2
c39e105 
Signed-off-by: Alec Scott <alec@llnl.gov>
@alecbcs
Update to use --yes-to-all for importing keys
b438ae4 
Signed-off-by: Alec Scott <alec@llnl.gov>
@alalazo @alecbcs
Update the basics section
6d11960 
Signed-off-by: Alec Scott <alec@llnl.gov>
@alalazo @alecbcs
Update the environments section
4f1a0cd 
Signed-off-by: Alec Scott <alec@llnl.gov>
@alecbcs
Add binary cache before installing keys
a44bd66 
Signed-off-by: Alec Scott <alec@llnl.gov>
@alecbcs
Keep gcc@14 in the container as an external compiler
33f48e5 
Signed-off-by: Alec Scott <alec@llnl.gov>
@alecbcs
Swap to spack packages v2026.06 release branch
e52aaeb 
Signed-off-by: Alec Scott <alec@llnl.gov>
@alalazo @alecbcs
Update the stacks section
74737c8 
Signed-off-by: Alec Scott <alec@llnl.gov>
@alecbcs
Add example environments from stacks rewrite
8f56a5a 
Signed-off-by: Alec Scott <alec@llnl.gov>
@alecbcs
Re-enable output generation in PR
15ecfa2 
Signed-off-by: Alec Scott <alec@llnl.gov>
alalazo and others added 3 commits June 19, 2026 10:40
@alalazo @alecbcs
Update the packaging section
44b460c 
Signed-off-by: Alec Scott <alec@llnl.gov>
@alecbcs
Revert packages.yaml config
64713b8 
Signed-off-by: Alec Scott <alec@llnl.gov>
@alecbcs
Merge branch 'main' into ci/isc26
077e55c
@alecbcs alecbcs changed the title Testing Bumping Container to 26.04 Re-enable Output Generation CI/CD on PRs (without writing back to PRs) Jun 19, 2026
@alecbcs alecbcs changed the title Re-enable Output Generation CI/CD on PRs (without writing back to PRs) Re-enable Output Generation CI/CD on PRs Jun 19, 2026
@alecbcs

alecbcs commented Jun 19, 2026

Copy link
Copy Markdown
Member Author

Force merging without another approval since most folks have gone to bed or are out for the holiday.

I've tested the CI in this PR fully and rebased off of the latest commits to main. It adds back the CI/CD check to ensure that future edits must pass the checks before being able to be merged so that we don't accidentally break the tutorial this weekend before giving it on Monday.

@alecbcs
alecbcs merged commit f87045b into main Jun 19, 2026
5 checks passed
@alecbcs
alecbcs deleted the ci/isc26 branch June 19, 2026 20:10
@kwryankrattiger

kwryankrattiger commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Caches are populated at s3://v2026.06.0/tutorial/

alecbcs added a commit that referenced this pull request Jul 21, 2026
@dependabot @alalazo @alecbcs
build(deps): bump docutils from 0.21.2 to 0.23 (#541)
df2c3ae 
* ISC26: update the basics section (#535)

* Extend gitignore

* basics: split "Installing Packages" into subsections

* Add a paragraph explaining the overall spack architecture

This is so that audience is already aware
of where we are headin into, when we introduce
the various pieces (hash, store, etc.).

* Update releases/v1.1 -> releases/v1.2

* Move high-level highlights to introduction

* Minor fixes

* Shorten and improve "Installing Packages"

* Demote Compiler flags

* Reorder "Spec Syntax" subsections

* Move "Querying Installations" into its own section

Also, consolidate all the use of `spack find`

* Add intro to "Spec Syntax"

* Update output

* Fix "Installing Packages"

* Fix "Transitive Dependencies"

* Fix "Transitive Dependencies"

* Fix "Virtual Dependencies"

* First draft

* Add missing output

* Polish the section

* Polish the section

* Temporarily disable output generation

* Minor style fixes

* Minor style fixes

* Address review

* Don't use "realistic"

* Remove old output files after merge with main

Signed-off-by: Alec Scott <alec@llnl.gov>

---------

Signed-off-by: Alec Scott <alec@llnl.gov>
Co-authored-by: Alec Scott <alec@llnl.gov>

* ISC26: update the environments section (#542)

* Extend gitignore

* basics: split "Installing Packages" into subsections

* Add a paragraph explaining the overall spack architecture

This is so that audience is already aware
of where we are headin into, when we introduce
the various pieces (hash, store, etc.).

* Update releases/v1.1 -> releases/v1.2

* Move high-level highlights to introduction

* Minor fixes

* Shorten and improve "Installing Packages"

* Demote Compiler flags

* Reorder "Spec Syntax" subsections

* Move "Querying Installations" into its own section

Also, consolidate all the use of `spack find`

* Add intro to "Spec Syntax"

* Update output

* Fix "Installing Packages"

* Fix "Transitive Dependencies"

* Fix "Transitive Dependencies"

* Fix "Virtual Dependencies"

* First draft

* Add missing output

* Polish the section

* Polish the section

* Temporarily disable output generation

* Mechanical update of the environment section

* Minor fixes

* Don't use a non-matching temporary environment

* Shorten introduction, but maintain the same content

* Improved "Working with environments" section

* Improved "Working with environments" section

* Added an "Environment on disk" section

This section introduces manifest and lockfile early.

* Added an "Environment on disk" section

This section introduces manifest and lockfile early.

* Added an "Environment on disk" section

This section introduces manifest and lockfile early.

* Added a "Configuring environments" section

* Trimmed redundant section

* First complete draft of the "Environments" tutorial

* Minor fixes

* Minor fixes

* Less notes in the initial section

---------

Co-authored-by: Alec Scott <alec@llnl.gov>

* ISC26: update the stacks section (#544)

* Extend gitignore

* basics: split "Installing Packages" into subsections

* Add a paragraph explaining the overall spack architecture

This is so that audience is already aware
of where we are headin into, when we introduce
the various pieces (hash, store, etc.).

* Update releases/v1.1 -> releases/v1.2

* Move high-level highlights to introduction

* Minor fixes

* Shorten and improve "Installing Packages"

* Demote Compiler flags

* Reorder "Spec Syntax" subsections

* Move "Querying Installations" into its own section

Also, consolidate all the use of `spack find`

* Add intro to "Spec Syntax"

* Update output

* Fix "Installing Packages"

* Fix "Transitive Dependencies"

* Fix "Transitive Dependencies"

* Fix "Virtual Dependencies"

* First draft

* Add missing output

* Polish the section

* Polish the section

* Temporarily disable output generation

* Mechanical update of the environment section

* Minor fixes

* Don't use a non-matching temporary environment

* Shorten introduction, but maintain the same content

* Improved "Working with environments" section

* Improved "Working with environments" section

* Added an "Environment on disk" section

This section introduces manifest and lockfile early.

* Added an "Environment on disk" section

This section introduces manifest and lockfile early.

* Added an "Environment on disk" section

This section introduces manifest and lockfile early.

* Added a "Configuring environments" section

* Trimmed redundant section

* First complete draft of the "Environments" tutorial

* Minor fixes

* Minor fixes

* Less notes in the initial section

* Update Stacks to Ubuntu 26.04

* Add spec groups

* Fix last nits

* Fix last nits

---------

Co-authored-by: Alec Scott <alec@llnl.gov>

* ISC26: update the packaging section (#547)

* Extend gitignore

* basics: split "Installing Packages" into subsections

* Add a paragraph explaining the overall spack architecture

This is so that audience is already aware
of where we are headin into, when we introduce
the various pieces (hash, store, etc.).

* Update releases/v1.1 -> releases/v1.2

* Move high-level highlights to introduction

* Minor fixes

* Shorten and improve "Installing Packages"

* Demote Compiler flags

* Reorder "Spec Syntax" subsections

* Move "Querying Installations" into its own section

Also, consolidate all the use of `spack find`

* Add intro to "Spec Syntax"

* Update output

* Fix "Installing Packages"

* Fix "Transitive Dependencies"

* Fix "Transitive Dependencies"

* Fix "Virtual Dependencies"

* First draft

* Add missing output

* Polish the section

* Polish the section

* Temporarily disable output generation

* Minor style fixes

* Minor style fixes

* Address review

* Don't use "realistic"

* ISC26: update the Packaging section

---------

Co-authored-by: Alec Scott <alec@llnl.gov>

* ISC26: update index and setup header (#548)

* Extend gitignore

* basics: split "Installing Packages" into subsections

* Add a paragraph explaining the overall spack architecture

This is so that audience is already aware
of where we are headin into, when we introduce
the various pieces (hash, store, etc.).

* Update releases/v1.1 -> releases/v1.2

* Move high-level highlights to introduction

* Minor fixes

* Shorten and improve "Installing Packages"

* Demote Compiler flags

* Reorder "Spec Syntax" subsections

* Move "Querying Installations" into its own section

Also, consolidate all the use of `spack find`

* Add intro to "Spec Syntax"

* Update output

* Fix "Installing Packages"

* Fix "Transitive Dependencies"

* Fix "Transitive Dependencies"

* Fix "Virtual Dependencies"

* First draft

* Add missing output

* Polish the section

* Polish the section

* Temporarily disable output generation

* Minor style fixes

* Minor style fixes

* Address review

* Don't use "realistic"

* ISC26: update index and setup

---------

Co-authored-by: Alec Scott <alec@llnl.gov>

* Re-enable Output Generation CI/CD on PRs (#543)

I've tested the CI in this PR fully and rebased off of the latest commits to main. It adds back the CI/CD check to ensure that future edits must pass the checks before being able to be merged so that we don't accidentally break the tutorial this weekend before giving it on Monday.

---------

Signed-off-by: Alec Scott <alec@llnl.gov>
Co-authored-by: Massimiliano Culpo <massimiliano.culpo@gmail.com>

* build(deps): bump actions/checkout from 4.1.7 to 7.0.0 (#551)

Bumps [actions/checkout](https://github.com/actions/checkout) from 4.1.7 to 7.0.0.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v4.1.7...9c091bb)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-version: 7.0.0
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* build(deps): bump pytest from 9.1.0 to 9.1.1 (#558)

Bumps [pytest](https://github.com/pytest-dev/pytest) from 9.1.0 to 9.1.1.
- [Release notes](https://github.com/pytest-dev/pytest/releases)
- [Changelog](https://github.com/pytest-dev/pytest/blob/main/CHANGELOG.rst)
- [Commits](pytest-dev/pytest@9.1.0...9.1.1)

---
updated-dependencies:
- dependency-name: pytest
  dependency-version: 9.1.1
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* build(deps): bump actions/download-artifact from 4.1.8 to 8.0.1 (#556)

Bumps [actions/download-artifact](https://github.com/actions/download-artifact) from 4.1.8 to 8.0.1.
- [Release notes](https://github.com/actions/download-artifact/releases)
- [Commits](actions/download-artifact@fa0a91b...3e5f45b)

---
updated-dependencies:
- dependency-name: actions/download-artifact
  dependency-version: 8.0.1
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* build(deps): bump docker/setup-qemu-action from 3.2.0 to 4.1.0 (#554)

Bumps [docker/setup-qemu-action](https://github.com/docker/setup-qemu-action) from 3.2.0 to 4.1.0.
- [Release notes](https://github.com/docker/setup-qemu-action/releases)
- [Commits](docker/setup-qemu-action@49b3bc8...0611638)

---
updated-dependencies:
- dependency-name: docker/setup-qemu-action
  dependency-version: 4.1.0
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* build(deps): bump docker/build-push-action from 6.9.0 to 7.2.0 (#553)

Bumps [docker/build-push-action](https://github.com/docker/build-push-action) from 6.9.0 to 7.2.0.
- [Release notes](https://github.com/docker/build-push-action/releases)
- [Commits](docker/build-push-action@4f58ea7...f9f3042)

---
updated-dependencies:
- dependency-name: docker/build-push-action
  dependency-version: 7.2.0
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* build(deps): bump docker/login-action from 3.3.0 to 4.2.0 (#552)

Bumps [docker/login-action](https://github.com/docker/login-action) from 3.3.0 to 4.2.0.
- [Release notes](https://github.com/docker/login-action/releases)
- [Commits](docker/login-action@9780b0c...650006c)

---
updated-dependencies:
- dependency-name: docker/login-action
  dependency-version: 4.2.0
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* build(deps): bump ruamel-yaml from 0.18.15 to 0.19.1 (#555)

Bumps ruamel-yaml from 0.18.15 to 0.19.1.

---
updated-dependencies:
- dependency-name: ruamel-yaml
  dependency-version: 0.19.1
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* ISC26: small typo and grammar fixes in env and stacks sections (#559)

Signed-off-by: Alec Scott <alec@llnl.gov>

* build(deps): bump isort from 7.0.0 to 8.0.1 (#557)

Bumps [isort](https://github.com/PyCQA/isort) from 7.0.0 to 8.0.1.
- [Release notes](https://github.com/PyCQA/isort/releases)
- [Changelog](https://github.com/PyCQA/isort/blob/main/CHANGELOG.md)
- [Commits](PyCQA/isort@7.0.0...8.0.1)

---
updated-dependencies:
- dependency-name: isort
  dependency-version: 8.0.1
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* build(deps): bump docutils from 0.21.2 to 0.23

Bumps [docutils](https://github.com/rtfd/recommonmark) from 0.21.2 to 0.23.
- [Changelog](https://github.com/readthedocs/recommonmark/blob/master/CHANGELOG.md)
- [Commits](https://github.com/rtfd/recommonmark/commits)

---
updated-dependencies:
- dependency-name: docutils
  dependency-version: '0.23'
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

---------

Signed-off-by: Alec Scott <alec@llnl.gov>
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Massimiliano Culpo <massimiliano.culpo@gmail.com>
Co-authored-by: Alec Scott <alec@llnl.gov>
Co-authored-by: Alec Scott <hi@alecbcs.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alalazo alalazo alalazo left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@alecbcs @kwryankrattiger @alalazo