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

5221 grafana url redirection #664

Closed
wants to merge 78 commits into from
Closed

Conversation

stalin4suse
Copy link
Contributor

Problem:
The following GitHub harvester/harvester#5221 has details of the problem.

Solution:
Added a redirect url to the nginx-config.yaml.

Related Issue:
Fixed in #5221

Test plan:
Manually tested this on Harvester by modifying the configmap and restarting the grafana pod.

bk201 and others added 30 commits June 5, 2023 14:25
Signed-off-by: Kiefer Chang <kiefer.chang@suse.com>
  patch nginx-config file with values from 102.0.1+up40.1.2

  fix the tar time check warnng

  fetch rancher/shell v0.1.18, both rancher-monitoring-crd and rancher-monitoring are using it

Signed-off-by: Jian Wang <w13915984028@gmail.com>
(cherry picked from commit 9d9576c)
(cherry picked from commit cd8f875)
Signed-off-by: Jian Wang <w13915984028@gmail.com>
(cherry picked from commit fc5d876)
Signed-off-by: masteryyh <yanhong.yang@suse.com>
(cherry picked from commit 2dbc7b5)
(cherry picked from commit 0540832)
Signed-off-by: Kiefer Chang <kiefer.chang@suse.com>
(cherry picked from commit dbdc1ad)
(cherry picked from commit 9702369)
Signed-off-by: Jian Wang <w13915984028@gmail.com>
(cherry picked from commit 48f3dde)
This reverts commit 5051b0d.

(cherry picked from commit 73619b4)
Signed-off-by: masteryyh <yanhong.yang@suse.com>
(cherry picked from commit 96d3094)
    - we need to update the /proc/cmdline for looks cos_state as root
    - use latest sle-micro baseOS image as dev branch
    - change some oem files that depend on the old root label

Signed-off-by: Vicente Cheng <vicente.cheng@suse.com>
(cherry picked from commit 08ad34a)

Conflicts:
	scripts/package-harvester-os
          - use v1.2 base image instead sle-micro-head
Signed-off-by: Kiefer Chang <kiefer.chang@suse.com>
(cherry picked from commit 1c872a8)
* addition of seeder addon

(cherry picked from commit e6027e5)
continued rework for install mode changes

working base for install mode with create and join

initial work for streaming disk

changed stream disk

trying out options for stream-disk

reverse order of console, update console during stream disk, resize LH partition on install

stream disk changes

fixed up partition labeling

re add generate rancherd config

improve image generation speed, and also updated validator changes

swap back consoles to original path

updated to leap 15.4 for qemu packages

refactored code to address codecov complexity failures, and disabled /dev/kvm in dapper to ensure current ci doesnt fail

added privileged permissions for dapper

(cherry picked from commit b934ba0)
(cherry picked from commit 1fc4eea)
(cherry picked from commit 94bcf3d)
(cherry picked from commit fcd524a)
    - change json format to yaml
    - skip G204 (gosec)
    - fix others lint errors

Signed-off-by: Vicente Cheng <vicente.cheng@suse.com>
(cherry picked from commit 1069a0c)
Removing some intermediate chart directories. We only need
tarballs in the chart repo.

Signed-off-by: Kiefer Chang <kiefer.chang@suse.com>
(cherry picked from commit f2f0cd4)
Signed-off-by: Kiefer Chang <kiefer.chang@suse.com>
(cherry picked from commit a7d4dbe)
Signed-off-by: masteryyh <yanhong.yang@suse.com>
(cherry picked from commit 60ad159)
    This is the side-effect from 1069a0c,
    the `Sign()` only return sign or not.

    Use wrapper function `GenRandNumber` to get the random number

Signed-off-by: Vicente Cheng <vicente.cheng@suse.com>
(cherry picked from commit 7027f89)
Connor Kuehl and others added 26 commits November 17, 2023 10:38
Signed-off-by: Connor Kuehl <connor.kuehl@suse.com>
(backported from commit cc7e7dc)
    We need guarantee the boot order like following:
    - `Patched tty service` -> `start tty service`

    So try to move the patch operation to the initramfs stage

Signed-off-by: Vicente Cheng <vicente.cheng@suse.com>
(cherry picked from commit 35dad50)
Signed-off-by: PoAn Yang <poan.yang@suse.com>
(cherry picked from commit bcd84ec)
Signed-off-by: PoAn Yang <poan.yang@suse.com>
(cherry picked from commit 21b69b1)
This commit adds disk size validation in the calls to diskConfirm()
and dataDiskConfirm(), so that as you walk through the fields you'll
see potential error messages as you go, rather than having to wait
until leaving the last field on the page.  The error message clears
if you press the up arrow to go to previous fields.

The persistent partition size check remains where it is, i.e. it's
only done when you try to move to the next page.  I couldn't figure
out a neat way to incorporate this into the validateAllDiskSizes()
function, which gets called before the default persistent size is
set in persistentSizeV.PreShow().

This commit also changes the wording of the "disk size is too small"
error messages to explicitly state *which* disk is too small, and
makes a small tweak to the "partition size" error message.

Signed-off-by: Tim Serong <tserong@suse.com>
(cherry picked from commit 9b6dd1e)
Signed-off-by: PoAn Yang <poan.yang@suse.com>
(cherry picked from commit d29437e)
Signed-off-by: PoAn Yang <poan.yang@suse.com>
(cherry picked from commit fae40bc)
Signed-off-by: PoAn Yang <poan.yang@suse.com>
Signed-off-by: PoAn Yang <poan.yang@suse.com>
(cherry picked from commit 87c2b5c)
    - Some firmware will make bootup stuck on the grub about 30 minutes.
      We could manually create these files to avoid seaching hang up.

Signed-off-by: Vicente Cheng <vicente.cheng@suse.com>
(cherry picked from commit 479d3c5)
Signed-off-by: Vicente Cheng <vicente.cheng@suse.com>
(cherry picked from commit 28fb14a)
Signed-off-by: Kiefer Chang <kiefer.chang@suse.com>
(cherry picked from commit ef5a34a)
Signed-off-by: Kiefer Chang <kiefer.chang@suse.com>
(cherry picked from commit c7f6b5d)
Signed-off-by: PoAn Yang <poan.yang@suse.com>
(cherry picked from commit 263dca7)
    - add `harvester.os.sshd.sftp` to config `sftp` dynamically

Signed-off-by: Vicente Cheng <vicente.cheng@suse.com>
(cherry picked from commit c3aab22)
Signed-off-by: PoAn Yang <poan.yang@suse.com>
(cherry picked from commit f54fbdd)
There's an off-by-one (or possibly off-by-two) error in the
current implementation of the select widget, when there's
content in the panel above it.  This means the last line of
text in the panel is overwritten by the select widget (i.e.
the select widget is one line too high).  This commit changes
the logic slightly to offset the select widget by the number
of lines of panel content plus one, allowing all the text
in the panel to be shown.

Signed-off-by: Tim Serong <tserong@suse.com>
(cherry picked from commit 19c750d)
  - drop the old elemental binary
  - drop luet
  - new elemental do not support download image by luet, we
    need to handle the boot related files manually
  - add debug flag when elemental build-iso

Signed-off-by: Vicente Cheng <vicente.cheng@suse.com>
(cherry picked from commit 6d76c36)
Signed-off-by: Kiefer Chang <kiefer.chang@suse.com>
(cherry picked from commit 1c42f30)
Signed-off-by: Kiefer Chang <kiefer.chang@suse.com>
(cherry picked from commit 6fe0734)
Signed-off-by: PoAn Yang <poan.yang@suse.com>
Signed-off-by: PoAn Yang <poan.yang@suse.com>
Signed-off-by: PoAn Yang <poan.yang@suse.com>
* silent rsync progress in harv-install

Because this is running without a TTY, it may be affecting IPMI-based installs that take hours to read the zst files off of virtual media.  This patch removes the progress flag from rsync in the hopes that it prevents rsync from freezing during install.

* restore -h and add -v flags

Human-readable and verbose flags added per developer request.

(cherry picked from commit d6947e0)
* Bump Rancher to v2.8.2
* Bump RKE2 to v1.26.13

---------

Signed-off-by: Kiefer Chang <kiefer.chang@suse.com>
@bk201
Copy link
Member

bk201 commented Mar 7, 2024

@stalin4suse @w13915984028 has referred your change and co-author you in the PR: #665, and we'll add you as a contributor in Harvester v1.3.0. Thanks!

@bk201 bk201 closed this Mar 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.