Remove broken dilos apt repo#156640
Conversation
CI is currently down because apt is unable to access the repo at dilos.org. It is failing inside the fortanix install script which tries to run apt-get with the error: ``` 5.679 E: Failed to fetch https://apt.dilos.org/dilos/dists/dilos2/InRelease 403 Forbidden [IP: 116.202.240.188 443] ``` These lines in the Dockerfile were added when Solaris builds were merged into dist-various-2 in rust-lang#45001. However, Solaris builds were split off in rust-lang#138699. In that PR, these lines adding the dilos repository weren't removed. AFAICT, these are no longer necessary.
|
r? @marcoieni rustbot has assigned @marcoieni. Use Why was this reviewer chosen?The reviewer was selected based on:
|
|
@bors treeclosed=100 |
|
Tree closed for PRs with priority less than 100. |
|
@bors r+ Should we kick off a try build or raise priority of this? |
|
@bors p=110 |
This comment has been minimized.
This comment has been minimized.
What is this?This is an experimental post-merge analysis report that shows differences in test outcomes between the merged PR and its parent PR.Comparing 3514361 (parent) -> be326a9 (this PR) Test differencesShow 2 test diffs2 doctest diffs were found. These are ignored, as they are noisy. Test dashboardRun cargo run --manifest-path src/ci/citool/Cargo.toml -- \
test-dashboard be326a93323e9b929525b9298f43883ac66b3b2d --output-dir test-dashboardAnd then open Job duration changes
How to interpret the job duration changes?Job durations can vary a lot, based on the actual runner instance |
|
Let's try reopening the tree. @bors treeopen |
|
Tree is now open for merging. |
|
Finished benchmarking commit (be326a9): comparison URL. Overall result: ❌ regressions - no action needed@rustbot label: -perf-regression Instruction countOur most reliable metric. Used to determine the overall result above. However, even this metric can be noisy.
Max RSS (memory usage)Results (secondary -2.7%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (primary 2.3%, secondary 2.3%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeThis perf run didn't have relevant results for this metric. Bootstrap: 513.812s -> 511.489s (-0.45%) |
|
Approving this for beta. I've asked @weihanglo to include it in #156625. |
CI is currently down because apt is unable to access the repo at dilos.org. It is failing inside the fortanix install script which tries to run apt-get with the error:
These lines in the Dockerfile were added when Solaris builds were merged into dist-various-2 in #45001. However, Solaris builds were split off in #138699. In that PR, these lines adding the dilos repository weren't removed.
AFAICT, these are no longer necessary.