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

1.0.10 Recording Importer Issue #471

Closed
semzor opened this issue Mar 2, 2021 · 3 comments · Fixed by #472
Closed

1.0.10 Recording Importer Issue #471

semzor opened this issue Mar 2, 2021 · 3 comments · Fixed by #472
Assignees
Labels
bug Something isn't working

Comments

@semzor
Copy link

semzor commented Mar 2, 2021

Importer doesn't import recordings from spool directory. Log related to this that I could find:

Failed to import recording: undefined method `casecmp?' for nil:NilClass

Do I need to explicitly set RECORDING_DISABLED=false in env file?

@wolbernd
Copy link

wolbernd commented Mar 2, 2021

Can confirm. Here is the full stacktrace:

W, [2021-03-01T11:08:50.490144 #6]  WARN -- : /srv/scalelite/lib/recording_importer.rb:12:in `import': undefined method `casecmp?' for nil:NilClass (NoMethodError)
        from /srv/scalelite/lib/tasks/recordings.rake:17:in `block (4 levels) in <main>'
        from /srv/scalelite/lib/tasks/recordings.rake:16:in `each'
        from /srv/scalelite/lib/tasks/recordings.rake:16:in `block (3 levels) in <main>'
        from /srv/scalelite/vendor/bundle/ruby/2.6.0/gems/listen-3.1.5/lib/listen/event/config.rb:23:in `call'
        from /srv/scalelite/vendor/bundle/ruby/2.6.0/gems/listen-3.1.5/lib/listen/event/processor.rb:115:in `_process_changes'
        from /srv/scalelite/vendor/bundle/ruby/2.6.0/gems/listen-3.1.5/lib/listen/event/processor.rb:19:in `block in loop_for'
        from /srv/scalelite/vendor/bundle/ruby/2.6.0/gems/listen-3.1.5/lib/listen/event/processor.rb:15:in `loop'
        from /srv/scalelite/vendor/bundle/ruby/2.6.0/gems/listen-3.1.5/lib/listen/event/processor.rb:15:in `loop_for'
        from /srv/scalelite/vendor/bundle/ruby/2.6.0/gems/listen-3.1.5/lib/listen/event/loop.rb:84:in `_wait_for_changes'
        from /srv/scalelite/vendor/bundle/ruby/2.6.0/gems/listen-3.1.5/lib/listen/event/loop.rb:42:in `block in setup'
        from /srv/scalelite/vendor/bundle/ruby/2.6.0/gems/listen-3.1.5/lib/listen/internals/thread_pool.rb:6:in `block in add'

@wolbernd
Copy link

wolbernd commented Mar 2, 2021

Small Update: The problem doesn't occur with v1.0.9 so going back to this version (at least for the recording importer) is a workaround.

@jfederico
Copy link
Member

@wolbernd yes, the issue is how that validation is implemented, if the variable does not exist it triggers the error. You should get it solved in 1.0.10 by adding that variable. RECORDING_DISABLED=false

@jfederico jfederico added the bug Something isn't working label Mar 2, 2021
@jfederico jfederico reopened this Mar 2, 2021
git-lama added a commit to git-lama/scalelite that referenced this issue Mar 2, 2021
git-lama added a commit to git-lama/scalelite that referenced this issue Mar 2, 2021
git-lama added a commit to git-lama/scalelite that referenced this issue Mar 2, 2021
git-lama added a commit to git-lama/scalelite that referenced this issue Mar 2, 2021
git-lama added a commit to git-lama/scalelite that referenced this issue Mar 2, 2021
git-lama added a commit to git-lama/scalelite that referenced this issue Mar 2, 2021
jfederico pushed a commit that referenced this issue Mar 3, 2021
berott added a commit to viralitygmbh/scalelite that referenced this issue Mar 28, 2021
* scalelite/v1.0-stable:
  SL-99: Fixed issue processing BBB23 recordings and mounted on NFS (blindsidenetworks#484)
  Add bug fix for issue blindsidenetworks#471 (blindsidenetworks#472)
  use load multiplier to calculate load
  SL-86: added new variables to README (blindsidenetworks#465)
  SL-91: scaelite-nginx has an error on start (blindsidenetworks#463)
  Change logic for calculating server load (blindsidenetworks#445)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants