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

Various documentation updates and fixes #492

Merged
merged 1 commit into from
Jan 29, 2021
Merged

Various documentation updates and fixes #492

merged 1 commit into from
Jan 29, 2021

Conversation

JaimeMagiera
Copy link
Contributor

@JaimeMagiera JaimeMagiera commented Jan 28, 2021

Updates to the FAQ, Known Issues, README.

@openshift-ci-robot openshift-ci-robot added needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. and removed needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Jan 28, 2021
FAQ.md Outdated
There is currently an issue with using Fedora CoreOS 33 for UPI installations. Please use the FCOS 32 image. After installation, the nodes will have rebooted and updated to the latest stable FCOS 33.
=======
There is currently an issue with using Fedora CoreOS 33 for UPI installations. Please use the FCOS 32 image. After installation, the nodes will have rebooted and updated to the latest stable FCOS 33. For example, you can use the [FCOS 32 .ova image](https://builds.coreos.fedoraproject.org/prod/streams/stable/builds/32.20201104.3.0/x86_64/fedora-coreos-32.20201104.3.0-vmware.x86_64.ova) for vSphere installations.
>>>>>>> 0cbed72... Various documentation updates and fixes

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please fix this rebase conflict

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, fixed.

KNOWN_ISSUES.md Outdated
**Description:** Cluster upgrades will fail due to incorrect image signatures.
**Workaround:** Perform the upgrade in the terminal using the --force flag
```bash
oc adm upgrade --to-latest=true --force=true

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you already tried to upgrade above command won't be enough.
Output will be like:

oc adm upgrade --to-latest=true --force=true
error: already upgrading.

  Reason: ImageVerificationFailed
  Message: Unable to apply 4.6.0-0.okd-2021-01-23-132511: the image may not be safe to use



If you want to upgrade anyway, use --allow-upgrade-with-warnings.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And then:

# oc adm upgrade --to-latest=true --force=true --allow-upgrade-with-warnings
warning: --allow-upgrade-with-warnings is bypassing: already upgrading.

  Reason: ImageVerificationFailed
  Message: Unable to apply 4.6.0-0.okd-2021-01-23-132511: the image may not be safe to use

warning: --force overrides cluster verification of your supplied release image and waives any update precondition failures.
Updating to latest version 4.6.0-0.okd-2021-01-23-132511

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Right, we should add oc adm upgrade --reset && oc adm update .. instead

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added --reset before forced upgrade and use of --allow-upgrade-with-warnings for forced upgrade.

Copy link

@sandrobonazzola sandrobonazzola left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See inline comments

Copy link

@sandrobonazzola sandrobonazzola left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jan 29, 2021
@sandrobonazzola sandrobonazzola removed their assignment Jan 29, 2021
FAQ.md Outdated
@@ -121,8 +121,7 @@ accessible and user workloads would be evicted and rescheduled as nodes are upda

## Should I use Fedora CoreOS 32 or 33 for installing on User Provisioned Infrastructure?

There is currently an issue with using Fedora CoreOS 33 for UPI installations. Please use the FCOS 32 image. After installation, the nodes will have rebooted and updated to the latest stable FCOS 33.

There is currently an issue with using Fedora CoreOS 33 for UPI installations. Please use the FCOS 32 image. After installation, the nodes will have rebooted and updated to the latest stable FCOS 33. For example, you can use the [FCOS 32 .ova image](https://builds.coreos.fedoraproject.org/prod/streams/stable/builds/32.20201104.3.0/x86_64/fedora-coreos-32.20201104.3.0-vmware.x86_64.ova) for vSphere installations.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think its still actual. F33 as a base should work fine.

Lets remove this and then re-add if folks complain?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Jan 29, 2021
Add link to Known Issues

adding reset and --allow-upgrade-with-warnings flag

fix merge

remove question on FCOS versions
Copy link
Member

@vrutkovs vrutkovs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!
/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jan 29, 2021
@vrutkovs vrutkovs merged commit 32e8772 into okd-project:master Jan 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants