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.

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

[rust] Automated Edge management (#11681 and #11683) #12835

Merged
merged 35 commits into from
Oct 7, 2023
Merged
Changes from 1 commit
Commits
Show all changes
35 commits
Select commit Hold shift + click to select a range
0668c6a
[rust] Automated Edge management (macOS) (#11681)
bonigarcia Sep 19, 2023
9a5ae46
[rust] Check also unstable versions for Edge management
bonigarcia Sep 19, 2023
501ffe6
[rust] Include logic to check fixed versions of Edge
bonigarcia Sep 19, 2023
b0d03e9
[rust] Include additional log messages
bonigarcia Sep 19, 2023
b277e8c
[rust] Get browser url again if empty
bonigarcia Sep 19, 2023
1d1a905
[rust] Include logic for stable label
bonigarcia Sep 20, 2023
64b282a
[rust] Automated Edge management (Linux) (#11681 and #11683)
bonigarcia Sep 24, 2023
1445fae
[rust] Fix paths used to extract edge
bonigarcia Sep 24, 2023
72ad82c
[rust] Clean extract label and fix searched version
bonigarcia Sep 27, 2023
12f9dab
[rust] Refactor logic for downloading browsers in a common function
bonigarcia Sep 28, 2023
21c23a9
[rust] Install Edge in Windows through the MSI installer
bonigarcia Sep 28, 2023
a55c408
[rust] Check admin permissions in Windows before downloading MSI inst…
bonigarcia Sep 28, 2023
b5b58b8
[rust] Include tests for managing Edge
bonigarcia Sep 28, 2023
6f35235
[rust] Use browser version in functions for requesting online repos
bonigarcia Sep 28, 2023
8f47db3
[rust] Refactor common logic when unavailable download or discovery
bonigarcia Sep 29, 2023
baa4651
[rust] Improve a couple of logging messages
bonigarcia Sep 29, 2023
90bbf8e
[rust] Fix condition checking Firefox nightly in mac
bonigarcia Sep 29, 2023
329abdf
[rust] Update cargo bazel lock fil
bonigarcia Sep 29, 2023
a508aac
[rust] Ensure empty parent path before moving files
bonigarcia Sep 29, 2023
1d71373
Revert "[rust] Ensure empty parent path before moving files"
bonigarcia Sep 29, 2023
c7565bb
[rust] Separate function to tnsure empty parent path before moving files
bonigarcia Sep 29, 2023
bb9f0d0
[rust] Revert function to create parent path in grid logic
bonigarcia Sep 29, 2023
514b29f
[rust] Fix logic to create empty parent path
bonigarcia Sep 30, 2023
2cf7a07
[rust] Fix problems after rebasing
bonigarcia Oct 6, 2023
efdc4f1
[ci] try without reusing docker in the tests
titusfortner Oct 6, 2023
3f2835c
[ci] run rbe with selenium manager in offline mode
titusfortner Oct 6, 2023
9fe3e52
[ci] set the input the correct way
titusfortner Oct 6, 2023
a35754d
try setting environment variable in bazelrc
titusfortner Oct 6, 2023
3fb7df2
turn off network on remote build
titusfortner Oct 6, 2023
42cb176
try setting the env in build as well as test
titusfortner Oct 6, 2023
cf27b9c
none of this worked
titusfortner Oct 7, 2023
5da0dc3
turn off selenium manager in test
titusfortner Oct 7, 2023
5c3f06a
fix linter issue
titusfortner Oct 7, 2023
8087dab
Merge branch 'trunk' into sm_edge_management
titusfortner Oct 7, 2023
8443a2b
Merge branch 'trunk' into sm_edge_management
titusfortner Oct 7, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
[rust] Check also unstable versions for Edge management
  • Loading branch information
bonigarcia committed Oct 6, 2023
commit 9a5ae46b1771da6df7eb7945ce0f16db041ec0d9
10 changes: 6 additions & 4 deletions rust/src/edge.rs
Original file line number Diff line number Diff line change
Expand Up @@ -414,7 +414,7 @@ impl SeleniumManager for EdgeManager {
// TODO check
fn request_latest_browser_version_from_online(&mut self) -> Result<String, Box<dyn Error>> {
let browser_version = self.get_browser_version();
let edge_updates_url = if browser_version.is_empty() {
let edge_updates_url = if browser_version.is_empty() || self.is_browser_version_unstable() {
BROWSER_URL.to_string()
} else {
format!("{}?view=enterprise", BROWSER_URL)
Expand Down Expand Up @@ -459,7 +459,10 @@ impl SeleniumManager for EdgeManager {
.unwrap()
.releases
.iter()
.filter(|r| r.architecture.eq_ignore_ascii_case(arch_label))
.filter(|r| {
r.platform.eq_ignore_ascii_case(os)
&& r.architecture.eq_ignore_ascii_case(arch_label)
})
.collect();
self.get_logger().trace(format!("Releases: {:?}", releases));

Expand All @@ -486,9 +489,8 @@ impl SeleniumManager for EdgeManager {
Ok(browser_version)
}

// TODO
fn request_fixed_browser_version_from_online(&mut self) -> Result<String, Box<dyn Error>> {
self.unavailable_download()
self.request_latest_browser_version_from_online()
}
}

Expand Down