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

Update failure #2914

Closed
gorkem opened this issue Apr 17, 2020 · 10 comments · Fixed by #3202
Closed

Update failure #2914

gorkem opened this issue Apr 17, 2020 · 10 comments · Fixed by #3202
Assignees
Labels
area/release-eng Issues or PRs related to the Release Engineering kind/bug Categorizes issue or PR as related to a bug.
Projects

Comments

@gorkem
Copy link
Contributor

gorkem commented Apr 17, 2020

/kind bug

I have tried to update my odo version using the recommended curl command on odo version command and it failed with the below messages.

Output of odo version:

❯ ❯ odo version
odo v1.0.0-beta5 (4d1d5071)

Server: https://api.rh-us-east-1.openshift.com:443
OpenShift: v3.11.154
Kubernetes: v1.11.0+d4cacc0
---
A newer version of odo (version: v1.1.2) is available.
Update using your package manager, or run
curl https://raw.githubusercontent.com/openshift/odo/master/scripts/installer.sh | sh
to update manually, or visit https://github.com/openshift/odo/releases
---
If you wish to disable the update notifications, you can disable it by running
'odo preference set UpdateNotification false'

❯ curl https://raw.githubusercontent.com/openshift/odo/master/scripts/installer.sh | sh
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  9213  100  9213    0     0  33501      0 --:--:-- --:--:-- --:--:-- 33501
# Detecting distribution...
# Detected platform: darwin-amd64
echo_stderr #
odo version "odo v1.0.0-beta5 (4d1d5071)" is already installed on your system. Running this installer script might cause issues with your current installation. If you want to install odo using this script, please remove the current installation of odo from you system.
Aborting now!
@openshift-ci-robot openshift-ci-robot added the kind/bug Categorizes issue or PR as related to a bug. label Apr 17, 2020
@girishramnani girishramnani added this to For consideration in Sprint 183 via automation Apr 20, 2020
@girishramnani girishramnani added area/component area/release-eng Issues or PRs related to the Release Engineering labels Apr 20, 2020
@kadel
Copy link
Member

kadel commented Apr 20, 2020

/remove-area component

@girishramnani girishramnani removed this from For consideration in Sprint 183 May 11, 2020
@girishramnani girishramnani added this to For consideration in Sprint 184 via automation May 11, 2020
@mohammedzee1000
Copy link
Contributor

The installer script needs an overall. I think we are already looking at breaking off into a sort of upstream/downstream so the install scripts are going to need an overhaul. cc @kadel

For now, i think the goal should be the script should detect if os is windows/mac/os (if memory serves bash is available on all of these os's) and the automatically curl download appropriate binary.

Long term though, assuming we break off, we should install from rpm for rpm based os's and do a curl download for rest. For downstream os'es we should instead ask use to subscribe to appropriate channels

If script fails, it will give link to binary

wdyt @girishramnani

@girishramnani
Copy link
Contributor

For release engineering, always go with long term solutions.

@girishramnani
Copy link
Contributor

Please plan for long term, what ever the solution might be

@mohammedzee1000
Copy link
Contributor

mohammedzee1000 commented May 13, 2020

Long term will need a decision on if we want to breakout upstream/downstream. Of course, we will still build internally, it the upload to github/internal delivery mechanisms and signing that will change. This is to avoid divergence as much as possible

@mohammedzee1000
Copy link
Contributor

One option we also have is dropping the script altogether and let our documentation handle it

@kadel
Copy link
Member

kadel commented May 14, 2020

One option we also have is dropping the script altogether and let our documentation handle it

I would be fine with that.

@mohammedzee1000
Copy link
Contributor

@gorkem considering its a simple matter of downloading binary and placing it in your path, it does not make sense to keep this script anymore. You can refer https://github.com/openshift/odo/releases and https://odo.dev/docs/installing-odo/.

@gorkem
Copy link
Contributor Author

gorkem commented May 18, 2020

That is fine as long as you are not printing a message that references a non-working script

@mohammedzee1000
Copy link
Contributor

That is happening in older version of odo. The newer versions do not point to the script

@mohammedzee1000 mohammedzee1000 moved this from To do to For review in Sprint 184 May 28, 2020
@girishramnani girishramnani added this to For consideration in Sprint 185 via automation Jun 1, 2020
@girishramnani girishramnani removed this from For review in Sprint 184 Jun 1, 2020
@girishramnani girishramnani moved this from For consideration to For review in Sprint 185 Jun 1, 2020
Sprint 185 automation moved this from For review to Done Jun 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/release-eng Issues or PRs related to the Release Engineering kind/bug Categorizes issue or PR as related to a bug.
Projects
No open projects
Sprint 185
  
Done
Development

Successfully merging a pull request may close this issue.

5 participants