Skip to content

Chore/remove python scripts#988

Merged
timcadman merged 6 commits intomasterfrom
chore/remove-python-scripts
Mar 17, 2026
Merged

Chore/remove python scripts#988
timcadman merged 6 commits intomasterfrom
chore/remove-python-scripts

Conversation

@timcadman
Copy link
Copy Markdown
Contributor

@timcadman timcadman commented Mar 17, 2026

To do:

  • Remove old python scripts from repository
  • Replace links in documentation to links of the scripts that are still on github

Test:

  • Old python scripts are gone
  • Dependabot doesn't show python packages in there anymore
  • Links in docs link to old scripts still on old version of armadillo on github

@timcadman timcadman marked this pull request as ready for review March 17, 2026 10:20
@timcadman timcadman linked an issue Mar 17, 2026 that may be closed by this pull request
5 tasks

Migration scripts referenced in this guide have been removed from the current repository as of Armadillo 5.13.2.
The scripts and their source code can be found in the
[Armadillo v3.4.0 release](https://github.com/molgenis/molgenis-service-armadillo/tree/v3.4.0/scripts/upgrade).
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

upgrade wasn't a thing back then, the scripts are just in the scripts folder:
https://github.com/molgenis/molgenis-service-armadillo/tree/v3.4.0/scripts


```bash
wget https://raw.githubusercontent.com/molgenis/molgenis-service-armadillo/master/scripts/upgrade/migrate-auth.py
wget https://raw.githubusercontent.com/molgenis/molgenis-service-armadillo/v3.4.0/scripts/upgrade/migrate-auth.py
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

### 1. Export Projects and users from Armadillo 2

To export users from an Armadillo 2 server, one must use the [export-users.py](https://github.com/molgenis/molgenis-service-armadillo/blob/master/scripts/upgrade/export-users.py) script. `export-users.py` can be used by using the following arguments:
To export users from an Armadillo 2 server, one must use the [export-users.py](https://github.com/molgenis/molgenis-service-armadillo/blob/v3.4.0/scripts/upgrade/export-users.py) script. `export-users.py` can be used by using the following arguments:
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

### 2. Import Projects and users TSVs into Armadillo 3

To import users into an Armadillo 3 server, one must use the [import-users.py](https://github.com/molgenis/molgenis-service-armadillo/blob/master/scripts/upgrade/import-users.py) script. `import-users` can be used by using the following arguments:
To import users into an Armadillo 3 server, one must use the [import-users.py](https://github.com/molgenis/molgenis-service-armadillo/blob/v3.4.0/scripts/upgrade/import-users.py) script. `import-users` can be used by using the following arguments:
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Collaborator

@marikaris marikaris left a comment

Choose a reason for hiding this comment

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

We have to check if dependapot is pleased after merge

@sonarqubecloud
Copy link
Copy Markdown

@timcadman timcadman merged commit 105e300 into master Mar 17, 2026
3 checks passed
@timcadman timcadman deleted the chore/remove-python-scripts branch March 17, 2026 11:12
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.

Remove old python scripts

2 participants