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.

Sign up for GitHub

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

Jump to bottom

Bump dependencies. #257

Merged
merged 2 commits into from
Jun 22, 2022
Merged

Bump dependencies. #257

merged 2 commits into from
Jun 22, 2022

Conversation

yakutovicha
Copy link
Member

  • Bump base image version to aiida-core:1.6.8
  • Bump aiidalab package to 22.02.0
  • Bump aiidalab-widgets-base to 1.3.3

WARNING: this changes the base image version to Ubuntu 20, which comes with a newer PostgreSQL. We do not yet have automated tools to migrate the PostgresSQL database.

@yakutovicha
Bump dependencies.
71b7b8e 
- Bump base image version to aiida-core:1.6.8
- Bump aiidalab package to 22.02.0
- Bump aiidalab-widgets-base to 1.3.3
@yakutovicha yakutovicha requested a review from csadorf June 21, 2022 10:27
@csadorf
Copy link
Member

csadorf commented Jun 21, 2022

WARNING: this changes the base image version to Ubuntu 20, which comes with a newer PostgreSQL. We do not yet have automated tools to migrate the PostgresSQL database.

This means this is blocked by #25 , correct?

@csadorf csadorf added the blocked This issue/PR is blocked by another issue/PR. label Jun 21, 2022
@yakutovicha
Copy link
Member Author

yakutovicha commented Jun 21, 2022 •
edited
Loading

WARNING: this changes the base image version to Ubuntu 20, which comes with a newer PostgreSQL. We do not yet have automated tools to migrate the PostgresSQL database.

This means this is blocked by #25 , correct?

I wouldn't block it, I would rather release an alpha version of the container so that Loris can work on it. FYI: aiida-core:1.6.8 comes with Python 3.8

@csadorf csadorf added this to the v22.07.01 milestone Jun 21, 2022
@csadorf
Copy link
Member

csadorf commented Jun 21, 2022 •
edited
Loading

WARNING: this changes the base image version to Ubuntu 20, which comes with a newer PostgreSQL. We do not yet have automated tools to migrate the PostgresSQL database.

This means this is blocked by #25 , correct?

I wouldn't block it, I would rather release an alpha version of the container so that Loris can work on it. FYI: aiida-core:1.6.8 comes with Python 3.8

Since I am worried that we might accidentally release a version that would require a migration, but comes without, I have created a milestone for the next release.

Can you update #25 and try to identify the next steps for it, please?

@yakutovicha
Copy link
Member Author

Can you update #25 and try to identify the next steps for it, please?

Just did it, please have a look.

@csadorf csadorf merged commit 8feadfb into develop Jun 22, 2022
@csadorf csadorf deleted the update/dependencies branch June 22, 2022 07:01
csadorf added a commit that referenced this pull request Jul 6, 2022
@csadorf
Roll-back to base image that does not require a PGSQL migration.
e629263 
This commit partially reverts #257.
@csadorf csadorf mentioned this pull request Jul 6, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@csadorf csadorf csadorf approved these changes

Assignees
No one assigned
Labels
blocked This issue/PR is blocked by another issue/PR.
Projects
None yet
Milestone
v22.07.01
Development

Successfully merging this pull request may close these issues.
2 participants
@yakutovicha @csadorf