* pass mb-edition to webpack build

* oops. pass "ee" rather than ":ee"

(Cannot choose granularity of dates on Summarize with Simple question from Saved Question)

[Fixes #7044] Render pulse bar charts with  negative values well

Also fixes misspelled column name in Avian Singles test dataset

* Bump DB prefixes so data gets reloaded to reflect col name change [ci drivers]

* Bump test DB version from v2 to v3

[ci drivers]

* More BigQuery test fixes 

 [ci drivers]

* Fix Snowflake [ci snowflake]

Co-authored-by: Cam Saul <github@camsaul.com>

(cherry picked from commit 4148885fd707b50750cefb339f994d7127dce702)

* Add a Postgres test [ci postgres]

* Mega-optimized SELECT probe query [ci drivers]

* Mega sync improvements [ci drivers]

* Even more amazing sync perf improvements [ci drivers]

* Fix bad edit [ci drivers]

* Test/lint fixes 

* Test fix  [ci drivers]

* Some driver fixes  [ci drivers]

* Address PR feedback [ci drivers]

* Test fixes for everything except for Oracle [ci drivers]

* Fix Oracle [ci drivers]

* Fix Redshift [ci redshift]

* Fix Pie chart visualization error when rows contain nils (#13626)

* Add positive assertion to CY test

* Fix prettier linter

* Quarantine the failing part of the "Pie chart" test

Co-authored-by: Nemanja <31325167+nemanjaglumac@users.noreply.github.com>

* Quarantine flaky tests (#13682) (#13683)

* Quarantine flaky tests (#13682)

* Add TODO note re: `describeWithToken`

Co-authored-by: Nemanja Glumac

* Add a circleci workflow that runs a set of Cypress smoketests (#12775)

* Add a circleci nightly workflow that runs a bigger set of Cypress smoketests

* Run this every hour while we're testing it, add a small fix to the admin test

* Smoketests: Skip some known failing tests

Co-authored-by: Ariya Hidayat <ariya@metabase.com>

* Narrow down the quarantine for chart_drill spec (#13682) (#13699)

* CI: Fix the cache key for uberjar for running smoketest (#13698)

Smoketest for OSS edition (for now)

* Fix `settings/settings.cy.spec.js` flakes (#13701)

- isolated flake (`it.only("should surface an error when validation for any field fails (metabase#4506)", () => {...})`) passed 10x (https://github.com/nemanjaglumac/metabase-tests/actions/runs/348275781)
- the whole spec passed 20x (https://github.com/nemanjaglumac/metabase-tests/actions/runs/348306177)

* CI: Update uberjar oss key for smoketest (#13704)

* Fix Cypress flakes in `dashboard/dashboard.cy.spec.js` file (#13703)

* Fix flakes in `revisions screen` block

* Improve test assertions (there were some rendering problems, so I adjusted the assertions)

* Narrow down the quarantine for dash_drill spec (#13682) (#13706)

* Fix `metrics.cy.spec.js` flakes (#13709)

* Fix `metrics.cy.spec.js` flakes (#13682)

* Refactor the test using API

* Smoketest: run nightly at 1am PST (#13711)

1 am PST = 09:00 UTC

CircleCI cron is for UTC:
https://circleci.com/docs/2.0/workflows/#scheduling-a-workflow

* Upgrade Cypress to v5.5.0 (#13674)

* Upgrade Cypress to v5.5.0

* Needed to unset the `DISPLAY` ENV through `config.yml` to finally make this work! (Reference: https://github.com/cypress-io/cypress/issues/4034#issuecomment-486773042

)

* Improve `segments` tests with the use of API (#13718)

* Unquarantine `scenarios/pulse/pulse.cy.spec.js` (#13727)

* Unset `DISPLAY` ENV in smoketests CI group (#13732)

* Unquarantine ee `custom_drill_through/drill_through.cy.spec.js` [ci skip] (#13734)

* Unquarantine `question/nested.cy.spec.js` (#13733)

Co-authored-by: Ariya Hidayat <ariya@metabase.com>
Co-authored-by: Damon P. Cortesi <d.lifehacker@gmail.com>

* Fix formatting temporal values as day of week with custom day of week (#13604)

* Pretty

* PR feedback

* Need to update when user locale changes as well

* Update Moment start of week in i18n.js instead

* Make it nice

* Remove some unneeded stuff

* Allow the Administrators group to sync

Allows the Administrators group to sync from LDAP or SAML providers.
This was previously not allowed to avoid a user accidentally locking
themselves out of the admin group.

Fixes metabase/metabase-enterprise#445

* test that admins are properly removed

Test that when a sync happens, if the user was previously in the administrators group and no longer is, they are properly removed from that group.

* Split expression suggestions into separate component
Fix error messages not appearing in ExpressionsWidget

* Enable test

* Fix suggestion popup keyboard behavior

* Don't try to parse empty input (#12140)

* Remove stray console.log

* Revert test enabling

* Prettier

* Address feedback

* Fingerprints reduce over table-rows-sample instead of realizing all

tests are failing because we test fingerprinters a lot by supplying
fake results and then reducing over those results. Now we are reducing
over them in the query processor and need to get a better testing
strategy.

I think i can stub out the query-processor to take the `:rff` and just
transduce over some fake supplied results but need to verify

* Hijack query-processor to use rff on fake data

* Docstring and make private `table-rows-sample-query`

* Fix tests to use query function rather than hijack qp

also was calling empty not empty? :(

* drivers check [ci drivers]

* Docstring cleanup for rff and remove outdated comment

* Make rff required for table-rows-sample

now you must reduce over the reducible row set rather than realizing
the results in memory and then working with them

* empty commit for drivers [ci drivers]

* Mega build script improvements

* Look for project.clj to determine root directory since we delete .git on CI for speed

* If .git folder is missing (i.e., in CI) fail gracefully

* Don't need to generate version file to run Cypress tests, right?

* Build uberjar step can cache version.properties
(so the Cypress test steps don't need to generate it again)

* Flush caches

* Update uberjar cache key

* Fix SQL for custom columns referring to aggregations. Fixes #12762

* Add test & enable cypress

* Rebase

* Typo

* Add repro for #10511

Co-authored-by: Nemanja <31325167+nemanjaglumac@users.noreply.github.com>

Enable the OOM killer, which will allow Docker / Kubernetes / ECS to restart a container when an OOM occurs

* Add repro for #13626 [ci skip]

Note: Once this issue is fixed, add a positive assertion to the Cypress test.

* upgrade some deps to fix warnings

* add back lint-yaml scrip

* remove e2e/karma references

* Add repro for #13235 [ci skip]
(Cannot filter by selecting remapped display value from dropdown list)

* Fix obsolete MBQL syntax in `localization.cy.spec.js` [ci skip]

Introduced in: https://github.com/metabase/metabase/commit/6b5765718daff384a1831cd7383401369b8c1725.

* Fix obsolete MBQL syntax in `dashboard.cy.spec.js` [ci skip]

Introduced in: https://github.com/metabase/metabase/commit/2958b836cc73c16b74b6e6e8fd96a6dabd12f60e.

* Fix obsolete MBQL syntax in `nested.cy.spec.js` [ci skip]

Introduced in: https://github.com/metabase/metabase/commit/ab762e1ffb8cd79fedc050709c059e8c8b87ef9b.

* Fix obsolete MBQL syntax in `dashboard-drill.cy.spec.js` [ci skip]

Introduced in: https://github.com/metabase/metabase/commit/cb4371e9fcc45ec242d106c4a01c4a0d68d4c324.

* Fix obsolete MBQL syntax in `nulls.cy.spec.js`

Introduced in: https://github.com/metabase/metabase/commit/854f5317df724abc0a6026b06bfa539d15d34fa6.

* Add positive assertion for repro for #13571

* Add repro for #12872 (Between Dates filter behaves inconsistently based on whether the column is from a joined table or not)

When a database is updated via the admin interface, any sensitive data
(private keys, passwords, etc) should be handled on an "upsert" basis -
only modify those fields if they were changed from the "protected
password" value.

This affects all database drivers.

Resolves #13442

* Fix collections/graph stack overflow

* Test fixes 

 [ci postgres] [ci mysql]

* Show engine in Task trouble shooting logs (#13608)

* Show engine in Task trouble shooting logs

in an effort to get better feedback when people raise issues about
long running sync processes, include the database engine in the logs.

Also wrapped `th` in a `tr` inside the `thead` to remove react warning

* Const intead of let on the db_id_to_engine

* Swap db id for db name

* Move comment out of jsx

* fixup! Merge branch 'release-x.37.x' into fix-collections-graph-stack-overflow

Co-authored-by: dpsutton <dan@dpsutton.com>

Merge master into release-x.37.x 10-29-2020

Migrate build/upload Elastic Beanstalk artifact code into main release script and other improvements (#13635)

* Always use current branch; determine CE vs EE build based on version number

* Rename delete-file! -> delete-file-if-exists!

* Dox & TODO message

* Move draft release template

* Some logging tweaks for deleting files

* Move s3 upload code to common namespace

* Integrate EB artifact logic

* Move .ebextensions -> release

* Remove old EB release scripts

* Update obsolete repro for #12985

* Remove `funnel_add` icon

* Put yaml-lint into devDependencies

That way, the CI run is more reproducible (no need to install yaml-lint
on the fly).

* Attempt to lint YAML files via yarn run

* Update the CI workflow dependency for yaml-linter

* Show engine in Task trouble shooting logs

in an effort to get better feedback when people raise issues about
long running sync processes, include the database engine in the logs.

Also wrapped `th` in a `tr` inside the `thead` to remove react warning

* Const intead of let on the db_id_to_engine

* Swap db id for db name

* Move comment out of jsx

* Fix failing Cypress test in `dashboard-drill` spec file

* Assert that the page content rendered (as per @dacort 's suggestion)

* [Fixes #11879] [Fixes #8410] Don't wrap email attachment filenames

* Non-ASCII characters in filenames are encoded [as per IETF RFC
  2047](https://tools.ietf.org/html/rfc2047) (c.f. #8410)
* Sufficiently-long _encoded_ filenames are sent over multiple lines
    as per [RFC 2184 section
    3](https://tools.ietf.org/html/rfc2184#section-3), for example:

    ```
    Content-Disposition: attachment;
      filename*0="=?UTF-8?Q?T=C3=A9=C5=BFting_non-ASCII_=C4=8D=C4=A5=C3=A4ract";
      filename*1="ers.xlsx?="
    ```
* Many mail clients do the right thing, but Gmail does not

This commit therefore turns off wrapping, as per [Bill Shannon's
Stack Overflow answer re. an undocumented system
property](https://stackoverflow.com/a/53344141/220529)
(Bill Shannon co-wrote the Java email library, which is used by postal
(the Clojure wrapper we use).

* (Hopefully) fix race condition in dashboard drill Cypress spec

Suggested by @flamber

* Skip flaky Cypress tests (per Nemanja)

* Only set `mail.mime.splitlongparameters` once

* Adds dev/src/dev/nocommit to ignore file [ci skip]

now you can create a folder nocommit/ under the dev classpath root so
you can keep some namespaces around as you work on features.

Ex:

```clojure
(ns dev.nocommit.oom
  (:require [metabase.models
             [database :refer [Database]]
             [field :refer [Field]]
             [table :refer [Table]]]
            [metabase.sync.analyze
             [fingerprint :as analyze.fingerprint]]
            [metabase.sync :as sync]
            [metabase.sync
             [field-values :as sync.field-values]]
            [metabase.models.field-values :as field-values]
            [metabase.sync.analyze.fingerprint.fingerprinters :as fingerprinters]
            [metabase
             [util :as u]]
            [metabase.sync.sync-metadata.fields :as metadata.fields]
            [metabase.sync.sync-metadata.fields.fetch-metadata :as fetch-metadata]
            [metabase.query-processor :as qp]
            [toucan.db :as db]
            [metabase.db
             [metadata-queries :as metadata-queries]]))

(def pg (Database 4))

(comment
  (fetch-metadata/db-metadata pg emails-table)
  (metadata.fields/sync-fields! pg)

  (sync/sync-database! pg))

(clojure.test/run-tests 'whatever-namespace)

```

I ended up with lots of `(comment ...)` blocks in lots of different
namespaces and this has helped keep my worktree clean.

* Empty commit to trigger CI [ci skip]

skipping ci thought so hopefully the checks on Github are satisified.