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

Deployment: Update the upgrade script #3492

Closed
ghost opened this issue Sep 3, 2018 · 11 comments
Closed

Deployment: Update the upgrade script #3492

ghost opened this issue Sep 3, 2018 · 11 comments

Comments

@ghost
Copy link

ghost commented Sep 3, 2018

As a developer, I want the upgrade script to be up to date so that I can install the latest version of IBEX on instruments more easily.

Include:

  • Tell the user to copy the third part installs from the share before installing.
  • Run the java panel updater as admin to check that auto-update is off.
  • Python does not always install (does it maybe use something local and then can't delete the python dir)
  • Sometime python can't find ps utils after restart is it using the wrong python in the script (should be using the network python and libraries)
  • Error should be produced if upgrade script fails to run
  • Got `Checking that configurations are being pushed to the appropriate repository
    Current repository status is: On branch NDXEMMA-A
    Your branch is up to date with 'origin/NDXEMMA-A'.

nothing to commit, working tree clean
Unexpected git status. Please confirm that configurations are being pushed to the appropriate remote repository` eevben though this is fine.

This came about while upgrading instruments to IBEX version 5.0.1 (see #3414)

@ghost ghost changed the title Update the upgrade script Deployment: Update the upgrade script Sep 3, 2018
@FreddieAkeroyd
Copy link
Member

The current upgrade script asks you to copy ibex_system_boot.bat into the “ProgramData” all users startup area. This requires administrator (gamekeeper) privilege and also means ibex would start if e.g. gamekeeper logged onto console itself.

I think a better approach would be to have ibex run as part of spudulike login only and also to use a shortcut in the startup area so the file does not need to be updated again if ibex_system_boot was to change in future.

So the new procedure would be:

  • Go to start -> all programs and right click on the startup folder and select “open”
  • In the directory do new -> shortcut and browse to c:\instrument\apps\epics and choose ibex_system_boot.bat

The upgrade script could check for this shortcut and create it if it is not present

@davidkeymer
Copy link
Contributor

The script should also install and configure Git, if required, as per the instructions on https://github.com/ISISComputingGroup/ibex_developers_manual/wiki/Getting-started-with-Git-and-GitHub

@davidkeymer
Copy link
Contributor

The script produces this output, which is now no longer correct.

Update web dashboard ...
Add the host name of the instrument to NDX_INSTS or ALL_INSTS in webserver.py in the JSON_bourne repository.```

@davidkeymer
Copy link
Contributor

Also wrong output:

"On NDAEXTWEB1, pull the updated code and add a link to the instrument dashboard on the main dataweb page under C:inetpub\wwwroot\Dat
aWeb\Dashboards\redirect.html"

@davidkeymer
Copy link
Contributor

davidkeymer commented Nov 1, 2018

As mentioned above, script provides confusing/incorrect instructions for creating shortcut to ibex_system_boot.bat in Startup Group of start menu.

Correct procedure here: https://github.com/ISISComputingGroup/ibex_developers_manual/wiki/Deployment-on-an-Instrument-Control-PC#creating-ibex-auto-startup

@John-Holt-Tessella
Copy link
Contributor

Add a step which selects a blank configuration or the top configuration in the list. After boot.

@LiamPanchaud
Copy link

LiamPanchaud commented Jan 29, 2019

  • Copy third party installs
  • Configure git
  • Add blank config
  • Change autostart location
  • Fix incorrect git error message
  • Remove web dashboard stuff
  • Error if script fails to run
  • Python can't find ps utils
  • Python does not always install
  • Run Java updater as admin

@LiamPanchaud
Copy link

@FreddieAkeroyd
Copy link
Member

Wiki page for ibex_system_boot.bat corrected

@ThomasLohnert
Copy link
Contributor

ThomasLohnert commented Jun 19, 2019

Some more issues I noticed in recent deployments:

  • "Upgrade Config" step tries to run misc/upgrade.bat without prompting for admin rights. This always fails and then has to be done manually
  • The script tries to be smart and automatically find the highest version number in the release folder. However this currently does not cater for installing from multiple release versions e.g. 5.4.1\Client_E4 and 5.4.0\EPICS - script would try to find both in 5.4.1. This is awkward to work around at the moment. We should make the script smarter and/or make it easier to manually pass in different release numbers for different system components

@Tom-Willemsen
Copy link
Contributor

Upgrade script has changed substantially since this issue was opened. Recent deployments have not had these issues.

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

No branches or pull requests

7 participants