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

[BUG] Obsolete theme:version config value of v8 (beta) rendering issues #3032

Open
kavilla opened this issue Dec 5, 2022 · 1 comment
Open
Assignees
Labels
bug Something isn't working

Comments

@kavilla
Copy link
Member

kavilla commented Dec 5, 2022

Describe the bug

The theme was marked hidden in Advanced Settings within the UI but is still can be utilized and configurable. Users who have originally set the theme version from v7 (or unset) to v8 (beta) experienced rendering issues within OpenSearch Dashboards. For example, bread crumbs would render like this on v8 (beta):

image

Reporting external plugin would render as:

image

Dashboards Version
2.x

Temp workaround
In 2.x version you can verify if your OpenSearch Dashboards utilized the v8 theme by running the following command in the Dev Tools, replacing $VERSION with your current version of the application:

GET .kibana/_doc/config:$VERSION

Even though the UI setting was removed in 2.x you can still update the setting by navigating to Dev Tools and pasting the following script:

POST .kibana*/_update_by_query
{
  "query": {
    "term": {
      "type": {
        "value": "config"
      }
    }
  },
  "script" : "ctx._source.config.remove('theme:version')"
}

Then refresh the application.


Curl commands of the above statement:

curl -XGET "http://localhost:9200/.kibana/_doc/config:$VERSION"

curl -XPOST "http://localhost:9200/.kibana*/_update_by_query" -H 'Content-Type: application/json' -d'
{
  "query": {
    "term": {
      "type": {
        "value": "config"
      }
    }
  },
  "script" : "ctx._source.config.remove('\''theme:version'\'')"
}'

Permanent solution
We should create a migration to remove this value automatically from the system index as this configuration should no longer be used.

s/o @AMoo-Miki, @manasvinibs, @joshuarrrr, @BSFishy for the help here.

@tmarkley
Copy link
Contributor

tmarkley commented Dec 7, 2022

Related: do we still want to address #494?

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

No branches or pull requests

4 participants