Skip to content

add update to apt-get for sqlite CI machine install #8924

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

Merged

Conversation

IanMatthewHuff
Copy link
Member

@IanMatthewHuff IanMatthewHuff commented Dec 5, 2019

For #8883

  • Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
  • Title summarizes what is changing
  • Has a news entry file (remember to thank yourself!)
  • Appropriate comments and documentation strings in the code
  • Has sufficient logging.
  • Has telemetry for enhancements.
  • Unit tests & system/integration tests are added/updated
  • Test plan is updated as appropriate
  • package-lock.json has been regenerated by running npm install (if dependencies have changed)
  • The wiki is updated with any design decisions/details.

@IanMatthewHuff IanMatthewHuff added the no-changelog No news entry required label Dec 5, 2019
@codecov-io
Copy link

codecov-io commented Dec 5, 2019

Codecov Report

Merging #8924 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #8924   +/-   ##
=======================================
  Coverage   58.16%   58.16%           
=======================================
  Files         529      529           
  Lines       27634    27634           
  Branches     4162     4162           
=======================================
  Hits        16073    16073           
  Misses      10671    10671           
  Partials      890      890

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9449a2e...d26195f. Read the comment docs.

@IanMatthewHuff
Copy link
Member Author

Context. On CI Linux machines our "Setup python to run with sqlite on 3.*" step has started failing recently. The first part of that step is to run the following:
sudo apt-get install libsqlite3-dev

A week or so ago on a working build:

The following NEW packages will be installed:
  libsqlite3-dev
0 upgraded, 1 newly installed, 0 to remove and 22 not upgraded.
Need to get 509 kB of archives.
After this operation, 1,708 kB of additional disk space will be used.
Get:1 http://azure.archive.ubuntu.com/ubuntu xenial-updates/main amd64 libsqlite3-dev amd64 3.11.0-1ubuntu1.2 [509 kB]

Last night the same step on a failing build:

The following NEW packages will be installed:
  libsqlite3-dev
0 upgraded, 1 newly installed, 0 to remove and 22 not upgraded.
Need to get 509 kB of archives.
After this operation, 1,708 kB of additional disk space will be used.
Err:1 http://security.ubuntu.com/ubuntu xenial-security/main amd64 libsqlite3-dev amd64 3.11.0-1ubuntu1.2
  404  Not Found
Err:1 http://security.ubuntu.com/ubuntu xenial-security/main amd64 libsqlite3-dev amd64 3.11.0-1ubuntu1.2
  404  Not Found
E: Failed to fetch http://security.ubuntu.com/ubuntu/pool/main/s/sqlite3/libsqlite3-dev_3.11.0-1ubuntu1.2_amd64.deb  404  Not Found

E: Unable to fetch some archives, maybe run apt-get update or try with --fix-missing?

Noting the comment on the end of the failure (and seeing that many azure apt-get steps listed online start out with an apt-get update. I've added an apt-get update before the install line.

@IanMatthewHuff IanMatthewHuff removed the no-changelog No news entry required label Dec 5, 2019
@IanMatthewHuff
Copy link
Member Author

Smoke Linux passes on my test build:
https://dev.azure.com/ms/vscode-python/_build/results?buildId=52963

The Install step now looks like this:

[command]/bin/bash --noprofile --norc /home/vsts/work/_temp/c012c483-8ee1-4d24-b980-d12282d95586.sh
Hit:1 http://azure.archive.ubuntu.com/ubuntu xenial InRelease
Get:2 http://azure.archive.ubuntu.com/ubuntu xenial-updates InRelease [109 kB]
<IANHU SNIP LIST>
Get:53 http://ppa.launchpad.net/ubuntu-toolchain-r/test/ubuntu xenial/main amd64 Packages [87.6 kB]
Get:54 http://ppa.launchpad.net/ubuntu-toolchain-r/test/ubuntu xenial/main Translation-en [14.6 kB]
Fetched 5,354 kB in 10s (535 kB/s)
Reading package lists...
Reading package lists...
Building dependency tree...
Reading state information...
The following additional packages will be installed:
  libsqlite3-0
Suggested packages:
  sqlite3-doc
The following NEW packages will be installed:
  libsqlite3-dev
The following packages will be upgraded:
  libsqlite3-0
1 upgraded, 1 newly installed, 0 to remove and 317 not upgraded.
Need to get 906 kB of archives.
After this operation, 1,708 kB of additional disk space will be used.
Get:1 http://azure.archive.ubuntu.com/ubuntu xenial-updates/main amd64 libsqlite3-0 amd64 3.11.0-1ubuntu1.3 [397 kB]
Get:2 http://azure.archive.ubuntu.com/ubuntu xenial-updates/main amd64 libsqlite3-dev amd64 3.11.0-1ubuntu1.3 [509 kB]

@IanMatthewHuff IanMatthewHuff changed the title add update to apt-get add update to apt-get for sqlite CI machine install Dec 5, 2019
@IanMatthewHuff IanMatthewHuff marked this pull request as ready for review December 5, 2019 19:03
@IanMatthewHuff IanMatthewHuff merged commit bb38bd9 into microsoft:master Dec 5, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Dec 12, 2019
@IanMatthewHuff IanMatthewHuff deleted the dev/ianhu/fixLinuxSQLite branch July 9, 2020 22:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants