Skip to content

fix: Fix various broken links in docs (#779) - #858

Merged
NickLarsenNZ merged 4 commits into
release-25.11from
fix/broken-links-25.11
Sep 9, 2026
Merged

NickLarsenNZ merged 4 commits into
release-25.11from
fix/broken-links-25.11

Conversation

@xeniape

@xeniape xeniape commented Sep 8, 2026

Copy link
Copy Markdown
Member

Description

Please add a description here. This will become the commit message of the merge request later.

Definition of Done Checklist

  • Not all of these items are applicable to all PRs, the author should update this template to only leave the boxes in that are relevant
  • Please make sure all these things are done and tick the boxes

Author

  • Changes are OpenShift compatible
  • CRD changes approved
  • CRD documentation for all fields, following the style guide.
  • Helm chart can be installed and deployed operator works
  • Integration tests passed (for non trivial changes)
  • Changes need to be "offline" compatible
  • Links to generated (nightly) docs added
  • Release note snippet added

Reviewer

  • Code contains useful comments
  • Code contains useful logging statements
  • (Integration-)Test cases added
  • Documentation added or updated. Follows the style guide.
  • Changelog updated
  • Cargo.toml only contains references to git tags (not specific commits or branches)

Acceptance

  • Feature Tracker has been updated
  • Proper release label has been added
  • Links to generated (nightly) docs added
  • Release note snippet added
  • Add type/deprecation label & add to the deprecation schedule
  • Add type/experimental label & add to the experimental features tracker
  • fix: Fix broken link to AirflowCluster CRD docs

  • broken link and external links

  • broken link and external links

* fix: Fix broken link to AirflowCluster CRD docs

* broken link and external links

* broken link and external links
@xeniape xeniape moved this to Development: Waiting for Review in Stackable Engineering Sep 8, 2026
@lfrancke

lfrancke commented Sep 8, 2026

Copy link
Copy Markdown
Member

CI fails

@NickLarsenNZ

Copy link
Copy Markdown
Member

CI fails

I took a look, and it seems like something is broken. An interpolation, or replacement.

Run curl --output /home/runner/.cargo/bin/ https://github.com/stackabletech/cargo-install-action/releases/download//-x86_64 --location --silent --show-error --fail --retry 5
curl: (22) The requested URL returned error: 404

I reran it, but same error. I'll see if I can figure it out.

@xeniape

xeniape commented Sep 8, 2026

Copy link
Copy Markdown
Member Author

CI fails

I took a look, and it seems like something is broken. An interpolation, or replacement.

Run curl --output /home/runner/.cargo/bin/ https://github.com/stackabletech/cargo-install-action/releases/download//-x86_64 --location --silent --show-error --fail --retry 5
curl: (22) The requested URL returned error: 404

I reran it, but same error. I'll see if I can figure it out.

When I check out an older release branch it usually complains about an old rust version in the toolchain file.
image
I just assume that's because we don't update that in older branches and that's why it's also failing on PRs. Was never relevant to me for the doc changes, so I ignored it for now. But of course might also be something different.

@NickLarsenNZ

Copy link
Copy Markdown
Member

But of course might also be something different.

This one is different.

Seems to be to do with inputs on this action:

https://github.com/stackabletech/cargo-install-action/blame/e3e2dcf8d0f0e5bdbc619bf6ee7560dd68152d3c/action.yml#L22-L23

@NickLarsenNZ

NickLarsenNZ commented Sep 8, 2026

Copy link
Copy Markdown
Member

My guess is that the inline interpolation is no longer allowed:

      run: curl --output ${{steps.cargo.outputs.dir}}/${{inputs.bin || inputs.crate}} https://github.com/stackabletech/cargo-install-action/releases/download/${{inputs.crate}}/${{inputs.bin || inputs.crate}}-${{steps.arch.outputs.arch}} --location --silent --show-error --fail --retry 5

It will instead need some env vars set, like:

      env:
        INPUT_CRATE: ${{inputs.crate}}
        INPUT_BIN: ${{inputs.bin || inputs.crate}}
      run: curl --output ${{steps.cargo.outputs.dir}}/ https://github.com/stackabletech/cargo-install-action/releases/download/$INPUT_CRATE/$INPUT_BIN-${{steps.arch.outputs.arch}} --location --silent --show-error --fail --retry 5

The rest seems to allow interpolations (ie: ${{steps.*}}).

@NickLarsenNZ

Copy link
Copy Markdown
Member

Fixed in #859

@NickLarsenNZ NickLarsenNZ moved this from Development: Waiting for Review to Development: In Review in Stackable Engineering Sep 9, 2026
@NickLarsenNZ
NickLarsenNZ merged commit 288c879 into release-25.11 Sep 9, 2026
16 of 17 checks passed
@NickLarsenNZ
NickLarsenNZ deleted the fix/broken-links-25.11 branch September 9, 2026 09:56
@NickLarsenNZ NickLarsenNZ moved this from Development: In Review to Development: Done in Stackable Engineering Sep 9, 2026
@lfrancke lfrancke moved this from Development: Done to Done in Stackable Engineering Sep 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

3 participants