-
Notifications
You must be signed in to change notification settings - Fork 14.7k
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 1.14 to Hugo 0.57.2 #16151
Update 1.14 to Hugo 0.57.2 #16151
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Error: Error building site: "/opt/build/repo/content/en/docs/tutorials/clusters/apparmor.md:126:1": failed to render shortcode "capture": failed to process shortcode: "/opt/build/repo/layouts/shortcodes/code.html:14:16": execute of template failed: template: shortcodes/code.html:14:16: executing "shortcodes/code.html" at <readFile $filename>: error calling readFile: runtime error: invalid memory address or nil pointer dereference
|
hello @zacharysarah, I do not fully understand the problem, but that file has one code shortcode. Will the file compile if you remove the shortcode, or change the shortcode to |
This compile error appears starting with Hugo v0.56.0, which had a lot of changes to the codebase. In the 1.16 master website code, neither the code nor codenew shortcodes could find deny-write.profile, which was in the same directory as apparmor.md (I tried several different things). To compile without error in Hugo v0.56+, I created a new directory: Then moved deny-write.profile from Finally I changed content/en/docs/tutorials/clusters/apparmor.md line 181 to use the codenew shortcode: Replicating this in the release-1.14 branch eliminates the compile error. However, in PR #16531, tengq replaces the shortcode in L181 with the contents of the deny-write.profile file, so I think it would be easier to cherry pick that change to earlier branches. However, part deux: The only shortcode in the blog post is {{< tweet >}}. This article explains under what circumstances the error is thrown and how to avoid it (increase timeout or store tweet json locally). Increasing the timeout value in config.toml eliminated the error on my local environment; most likely not needed in CI/CD environment. Now 1.14 compiles under Hugo 0.57.2. However, part 3: The blog R menu - the year doesn't expand/collapse when I click on it (Firefox 69, Chromium 76 on Ubuntu 18) |
3899f7e
to
a44047f
Compare
Closing/reopening this PR due to merge wonkiness. |
/close |
@zacharysarah: Closed this PR. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Modify apparmor.md and delete .profile to enable the site to build with Hugo 0.57.2. Could not cherry pick PR 16531 because the Korean translation was added in 1.14. See also PRs #16104 and #16151. zh/docs/reference/setup-tools/kubeadm/generated/README.md updated to fix 'invalid YAML delimiter' compile error caused by HTML comment. Signed-off-by: Aimee Ukasick <aimeeu.opensource@gmail.com>
This is the 1.14 version of #16104.