2872: Merge branch 'master' into 2872-folder-nav

Conflicts:
apps/workbench/app/assets/javascripts/pipeline_instances.js
apps/workbench/app/controllers/application_controller.rb
apps/workbench/app/controllers/collections_controller.rb
apps/workbench/app/controllers/jobs_controller.rb
apps/workbench/app/helpers/application_helper.rb
apps/workbench/app/models/job.rb
apps/workbench/app/views/application/_content.html.erb
apps/workbench/app/views/application/_show_metadata.html.erb
apps/workbench/app/views/pipeline_instances/_show_components.html.erb

2872: Fix bookmark bar causing spurious window width.

refs #2871
Merge branch '2871-preload-objects'

2871: fix the typo caught by integration test

closes #2871
Merge branch '2871-preload-objects'

Bugfix: the discovery doc should not clobber defaults values with the
word 'string'. Instead, make sure the default values are always exposed
as a string, which is something the java google apiclient is sensitive
to.

Refs no issue #

Added error-handler for x-editable that extracts Arvados API error field for
display, closes #2819

2871: add andand to prevent error

Merge branch 'master' into 2871-preload-objects

Merge branch '2961-load-tab-partials' closes #2961

arv-run-pipeline-instance should not keep trying to create jobs when the
API server returns an error on job creation.

This closes #3005

Merge branch '2826-simple-go-sdk' closes #2826

Fix inconsistent JSON error return behavior on job create.

refs #3004

Merge branch '2880-crunch-dispatch-node-constraints'

Closes #2880, #2976, #2993.

2880: Don't dispatch Jobs until runtime constraints are met.

This retains the same FIFO approach to the Job queue that
crunch-dispatch currently uses, but now when it encounters a Job whose
constraints are not met:

* it may wait for a while to see if the Node Manager makes Nodes
  available, if it hasn't done that this hour; and

* it leaves that Job in the queue and tries to process the next one.

See #2880 for further background.  The exact parameters of "waiting
for Nodes" will probably need tuning, but that will be easier to do
after it's been in production for a while.

2880: Improve error reporting when crunch-dispatch updates node status.

2880: crunch-dispatch must stop putting symbols in the database.

2961: Removed calls to wait_for_ajax, fixes user tests.

2961: user-setup-modal-window div is no longer in footer_html because it is
loaded using AJAX.

2961: Don't call #render_index from #index on folders_controller because #index
is also used by #choose and the index does not have any tabs.

2961: Do not include meta tag for websockets if there is no websockets in the
discovery document.

2961: Fixed name of collection on collection show page to fix test.

2961: Track valid/up to date state for each panel, only reload panels when they
are out of date (based on event log) and visible to user.

2961: Added use count so ajax loading icon is accurate when there are multiple
outstanding ajax requests.

2961: Removed dangling close div.

2961: Added missing user uuid to session.

Merge branch 'master' into 2961-load-tab-partials

Fix for running workbench against server that doesn't have 'Job.supplied_script_version' yet.  refs #2939

2961: Job index needs to call super sometimes, render_index other times.

2961: Changed index methods on some controllers to use render_index instead of
super, to skip the default index query logic but still apply the tab partial
logic.

Merge branch 'master' into 2871-preload-objects

2871: trailing whitespace

Specify server-side keep-alive ping interval for websockets.  no issue #

2961: Escaped erb used embedded in javascript.  Updated comments in
event_log.js.  Removed debug put from ApplicationController.

Merge branch 'master' into 2871-preload-objects

2871: test preload_objects_for_dataclass helper method

Add puma to the Gemfile for the API server; we use it for the websockets
server.

no issue #

2871: preload_log_collections tests

2871: more testing

2871: more tests

2871: test invoking methods a second time to retrieve preloaded data

2871: add tests for get_n_objects_of_class helper method

2961: Collections show controller now delegates to super to handle tabs
properly.

2871: add tests for preload_collections_for_objects helper method

2871: add input checks for helper methods. start adding functional tests for the helper methods.

Merge remote-tracking branch 'origin/master' into 2961-load-tab-partials

2961: Save user information as part of the session cookie, eliminates looking
up current user when rendering every page and panel, this means 2x reduction in
latency when the panel only requires 1 API lookup otherwise.

2872: Fix test for new nav.

2872: Top nav gradient.

2872: Hide projects nav bar when not logged in or inactive.

2872: Fix notifications menu colors.

2872: Rearrange top nav with breadcrumbs/projects bar.

2872: Rename folders to projects

2872: Hide unnecessary scroll bar

Merge branch 'master' into 2871-preload-objects

2872: updates to the new test that creates and runs pipeline instance from within a folder.

2872: added a test that creates a pipeline instance from within a folder and runs it.

2872: Fix provenance graph absolute position.

2872: add test for a pipeline instance with job associated so that the provenance graph can be tested.

2872: test now passing since the UI issue with "Resume" button is addressed.

Merge branch '2857-collection-tests'

Closes #2857.

Merge branch '2872-folder-nav' into 2872-folder-nav-tests

2872: Fix string+nil bug.

2872: Fix up pipeline instance table, show "resume" button if paused.

Merge branch 'master' into 2871-preload-objects

2872: added graph and run button checks back. the test does not pass right now due to bug in graph tab display in pipeline instance display.

2857: strip signatures from manifests before testing equality.

As Brett observed, the "post" method in these tests mutates its
argument. For a valid test, we must ensure that the source data has not
been transformed into the result data.

For tests which do not care about validating the signatures, we just
strip out the signatures from the result manifest before comparing it to
the source.

Refs #2857.

Merge branch '2872-folder-nav' into 2872-folder-nav-tests

2872: Bring back Graph pane on pipeline instance show page.

2872: when a pipeline instance still needs input, the Run button should be disabled.

Merge branch '2872-folder-nav' into 2872-folder-nav-tests

2872: updated test to work with the new ui.

2961: Added checks if we are rendering a tab partial and skips expensive calls
to the API server if not needed to render the requested tab pane.

2872: Fix auto-disable/enable "Run" button on pipeline page.

2872: On front page, only show recent pipelines and jobs that are in folders.

Merge branch 'master' into 2857-collection-tests

Merge branch '2955-fail-orphan-jobs' closes #2955

Reverted change a0aba6d "crunch-dispatch now sends a clean environment to crunch-job"

Merge branch 'master' into 2955-fail-orphan-jobs

Reverted change a0aba6d "crunch-dispatch now sends a clean environment to crunch-job" refs #2882.

Merge branch '2880-compute-ping-stats'

Closes #2967, #2975.

2880: API server saves node statistics from pings.

Crunch will use this information when scheduling Jobs, to satisfy
specified runtime constraints.  Getting this data into the pings is
the responsibility of the node's ping script.

2882: Workbench generic create controller adds JSON href attribute.

This is a bugfix commit.  This detail got lost when create stopped
calling show to render the object.  Refs #2882.

2872: update test to meet the current ui updates. the pipeline instances test still failing; possible error in ui where the Run button is enabled even when input is not provided.

Merge branch '2872-folder-nav' into 2872-folder-nav-tests

2872: Use collection name, not hash, for breadcrumbs/title/activity.

2872: Enable OK/Cancel buttons on editable popup. Lose "close" button.

2872: Tweak wording on pipeline inputs page.

2872: Fix up login behavior.

2872: Make "inactive" and "not logged in" behavior work on all pages, not just dashboard.

2872: Ensure user agreement can be viewed before being accepted.

2872: all tests are updated to reflect the new ui.

2961: Now loads tab panes (past the first one) in the background.
Automatically triggers reload of tab panes when receiving a changed event from
websocket.

2961: Removed spurious elementId from subscribeToEventLog(). Added arv-log-event-listener[data-object-type="xxxxx"] to
match on uuid types.

Fixes PipelineInstancesControllerTest.  application_controller#create now does
the right thing based on requested response format.  refs #2882

2857: fix flaky time-sensitive tests

Allow tests to inject a mock Time object into Blob, in order to generate
consistent behavior for tests that rely on permission signatures.

Closes #2857.

2872: update tests to match the new ui

Merge branch '2882-job-process-stats' refs #2882

2955: Renamed script to "cancel_stale_jobs".  Fixed spelling canceled_at -> cancelled_at.

2872: fix test