Skip to content

Refresh APT package index before installing Ubuntu build dependencies - #282

Open
ruokun-niu wants to merge 1 commit into
mainfrom
build-from-source-ubuntu
Open

ruokun-niu wants to merge 1 commit into
mainfrom
build-from-source-ubuntu

Conversation

@ruokun-niu

Copy link
Copy Markdown
Contributor

Description

Adds sudo apt-get update before dependency installation in the Debian/Ubuntu build-from-source guide to avoid package-download 404 errors caused by stale package indexes.

The verification workflow automatically uses the updated documented commands.

Type of change

  • This pull request fixes a bug in Drasi and has an approved issue (issue link required).
  • This pull request adds or changes features of Drasi and has an approved issue (issue link required).
  • This pull request is a minor refactor, code cleanup, test improvement, or other maintenance task and doesn't change the functionality of Drasi (issue link optional).

Fixes: #281

Signed-off-by: Ruokun Niu <ruokunniu@microsoft.com>
Copilot AI balanced review requested due to automatic review settings October 27, 2026 20:22

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The single-line change is correctly placed, preserves workflow compatibility, and has no identified blocking issues.

No open issues 🎉

What changed in this PR

Updates the Debian/Ubuntu build-from-source prerequisites to refresh the APT package index before installing dependencies, addressing stale-index download failures reported in #281.

Changes:

  • Adds sudo apt-get update immediately before dependency installation in the command block consumed by the existing verification workflow.
File Description
docs/​shared-content/​installation/​drasi-server/​build-from-source-prereqs.md Refreshes the APT index before installing native build dependencies.

Review effort: Balanced

@ruokun-niu
ruokun-niu marked this pull request as ready for review October 27, 2026 22:24
@ruokun-niu

Copy link
Copy Markdown
Contributor Author

With drasi-project/drasi-core#1000 being merged, we need to do a new crates release and a drasi server version bump

This branch has not been deployed

No deployments
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.

Scheduled "Verify Build from Source" run failed

4 participants