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

idevicesyslog managed by appium when not requested #10849

Closed
kazzmir opened this issue Jun 11, 2018 · 6 comments
Closed

idevicesyslog managed by appium when not requested #10849

kazzmir opened this issue Jun 11, 2018 · 6 comments
Assignees
Labels
Enhancement feature XCUITest regarding xcuitest driver

Comments

@kazzmir
Copy link

kazzmir commented Jun 11, 2018

The problem

The appium-ios-driver will 1) kill any existing idevicesyslog processes for the given udid being tested (https://github.com/appium/appium-ios-driver/blob/30b3c545278cba286b786c8360812164cf7d5f3e/lib/device-log/ios-log.js#L279) and 2) spawn a new idevicesyslog even if the showIOSLog capability is not set.

Environment

  • Appium version (or git revision) that exhibits the issue: 1.8.1
  • Desktop OS/version used to run Appium: macos
  • Mobile platform/version under test: ios
  • Real device or emulator/simulator: real

Details

I run my own idevicesyslog but it is killed by appium when a new session is started. I would prefer that appium does not kill my idevicesyslog process. My current workaround is to copy the idevicesyslog binary to a new name, thus avoiding being targeted by pkill.

@mykola-mokhnach mykola-mokhnach added Enhancement feature XCUITest regarding xcuitest driver labels Jun 11, 2018
@jlipps
Copy link
Member

jlipps commented Jun 12, 2018

@mykola-mokhnach @imurchie do we need to run idevicesyslog for our own purposes even if log capture is not requested via caps?

@mykola-mokhnach
Copy link
Collaborator

Previously it was necessary because we were reading WDA URL from the log, but now it is not the case anymore. I think we could add a separate capability to not start the log listener automatically. However, this will require to change several other places where we assume that the log listener is running.

@jumois
Copy link

jumois commented Dec 13, 2018

I started to use deviceconsole

I think we could add a separate capability to not start the log listener automatically.

This would be great. idevicesyslog has high CPU usage compared to deviceconsole.

@KazuCocoa
Copy link
Member

@KazuCocoa
Copy link
Member

skipLogCapture capability can disable crash, system, safari console and safari network logs by appium/appium-xcuitest-driver#878

@lock
Copy link

lock bot commented Feb 28, 2020

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked and limited conversation to collaborators Feb 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Enhancement feature XCUITest regarding xcuitest driver
Projects
None yet
Development

No branches or pull requests

5 participants