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

Broken template handling between 0.53 and 0.57.2 #6277

Closed
zacharysarah opened this issue Aug 29, 2019 · 6 comments
Closed

Broken template handling between 0.53 and 0.57.2 #6277

zacharysarah opened this issue Aug 29, 2019 · 6 comments
Labels

Comments

@zacharysarah
Copy link

We're currently running https://kubernetes.io on Hugo 0.53.

We noticed behavior similar to #5615 when the deprecationwarning.html shortcode renders in deprecated versions. Consequently, we opened a pull request to upgrade Hugo from 0.53 to 0.57.2.

Upgrading to 0.57.2 fixes the #5615 behavior, but breaks template handling for all pages across the site.

Note that k/website:#16104 changes only the site's Hugo version.

For comparison:

Production build (0.53) Preview build (0.57.2)
The Kubernetes API The Kubernetes API
@bep
Copy link
Member

bep commented Aug 29, 2019

This is related to https://github.com/gohugoio/hugo/releases/tag/v0.55.0

I'm pretty sure that I created a PR in your repo with a backward-compatible patch. I don't think that PR was merged.

In any case, the newer Hugo version is tested to work fine with the Kubernetes site.

@bep bep closed this as completed Aug 29, 2019
@bep bep reopened this Aug 29, 2019
@zacharysarah
Copy link
Author

@bep

I'm pretty sure that I created a PR in your repo with a backward-compatible patch. I don't think that PR was merged.

It looks like the backwards-compatible fixes did merge, but you asked us to delay upgrading to 0.55.3.

In any case, the newer Hugo version is tested to work fine with the Kubernetes site.

Given the extent of breakage in previews for a PR k/website:16104 that only upgrades the Hugo version, this doesn't seem quite accurate.

@bep
Copy link
Member

bep commented Aug 29, 2019

that only upgrades the Hugo version, this doesn't seem quite accurate.

Before I released Hugo 0.55 I diffed the output of the Kubernetes site before/after and they came out identical.

I just tested the Kubernetes site with the version in kubernetes/website#13680 with the latest Hugo and it works fine. You're right that it doesn't work with the current Kubernetes master, but that must just mean that someone has somehow undone the fixes that I applied.

In any case, I don't think we can do anything about it on the Hugo side of it, so I suggest you continue this discussion on the Kubernetes issue tracker.

@zacharysarah
Copy link
Author

that must just mean that someone has somehow undone the fixes that I applied.

It turns out we needed to cherry-pick the commit into the release-1.13 branch. We only merged the 0.55 backwards compatibility commit to master when it pointed to release-1.14 and later.

So, you are correct and all is good now. 🙇

@bep
Copy link
Member

bep commented Aug 30, 2019

Great. Now that you're getting the Hugo version up to date, you may want to consider using the new Hugo Modules to split the website and content into versionable modules, and possibly also split content per language, to possibly get something that's easier to manage.

@github-actions
Copy link

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants