Skip to content

Conversation

kklein
Copy link
Collaborator

@kklein kklein commented Jul 25, 2022

msodbcsql17
0 upgraded, 0 newly installed, 1 downgraded, 0 to remove and 13 not upgraded.
E: Packages were downgraded and -y was used without --allow-downgrades.
Error: Process completed with exit code 100.

See e.g. here [0] for current problem in CI.

[0] https://github.com/Quantco/datajudge/runs/7494860116?check_suite_focus=true

@kklein kklein requested a review from ivergara July 25, 2022 07:15
@kklein kklein marked this pull request as ready for review July 25, 2022 07:16
@jonashaag
Copy link
Contributor

Interesting. If we already have a version of msodbcsql17 installed, why even install another version?

@kklein
Copy link
Collaborator Author

kklein commented Jul 25, 2022

My interpretation was that some other dependency would be downgraded, not msodbcsql17 itself - I'm quite unsure about this, though.

@jonashaag
Copy link
Contributor

No:

The following packages will be DOWNGRADED:
  msodbcsql17

Can you try removing the entire msodbcsql17 install/upgrade/downgrade?

@kklein
Copy link
Collaborator Author

kklein commented Jul 25, 2022

I'd like to think that our docker image simply comes with the driver already installed. Yet, I'm not sure why we added this installation step.

I found a dockerfile installing msodbcsql18:

https://github.com/microsoft/mssql-docker/blob/master/linux/mssql-tools/Dockerfile.ubuntu2004#L15

I think I would be in favor of sticking with the downgrade for now and delaying the decision as to whether to get rid of the installation step until @ivergara is back. Does that sound good to you @jonashaag ?

@jonashaag
Copy link
Contributor

Sure.

@kklein kklein merged commit c84d6a6 into main Jul 25, 2022
@kklein kklein deleted the fix_ci branch July 25, 2022 09:18
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.

2 participants