-
Notifications
You must be signed in to change notification settings - Fork 55
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
Example App with automatic tailored service account setup #203
Merged
Merged
Changes from 2 commits
Commits
Show all changes
21 commits
Select commit
Hold shift + click to select a range
fed7c84
Add a plugin example with automatic service account setup
gamab aec36d7
Update readme
gamab b3bb30e
Merge remote-tracking branch 'origin/main' into gamab/appwithservicea…
gamab 0710a8f
Update example with latest changes
gamab 6985221
forgot to update the go.mod
gamab d6ed366
Use bearer token middleware
gamab 3984d56
Token on top of response
gamab 9efc417
Changing image
gamab ab6a205
Use custom header
gamab ed8955c
Revert change on OnBehalf plugin
gamab f30d154
Copy fail
gamab 7b50579
Remove unecessary horizontal group
gamab 78c50a3
Change screenshot again
gamab 34382db
Update the README
gamab ae40e87
modify docker compose to use 10.2
gamab eeed747
Copy package-lock.json from other example
gamab 050bc60
Account for PR feedback
gamab e067c50
rm comma
gamab 4b6d9e6
rm comma
gamab 2df32b8
Add PUT and more permissions
gamab 49b5e76
README update
gamab File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,13 @@ | ||
/* | ||
* ⚠️⚠️⚠️ THIS FILE WAS SCAFFOLDED BY `@grafana/create-plugin`. DO NOT EDIT THIS FILE DIRECTLY. ⚠️⚠️⚠️ | ||
* | ||
* In order to extend the configuration follow the steps in | ||
* https://grafana.github.io/plugin-tools/docs/advanced-configuration#extending-the-eslint-config | ||
*/ | ||
{ | ||
"extends": ["@grafana/eslint-config"], | ||
"root": true, | ||
"rules": { | ||
"react/prop-types": "off" | ||
} | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,16 @@ | ||
/* | ||
* ⚠️⚠️⚠️ THIS FILE WAS SCAFFOLDED BY `@grafana/create-plugin`. DO NOT EDIT THIS FILE DIRECTLY. ⚠️⚠️⚠️ | ||
* | ||
* In order to extend the configuration follow the steps in .config/README.md | ||
*/ | ||
|
||
module.exports = { | ||
"endOfLine": "auto", | ||
"printWidth": 120, | ||
"trailingComma": "es5", | ||
"semi": true, | ||
"jsxSingleQuote": false, | ||
"singleQuote": true, | ||
"useTabs": false, | ||
"tabWidth": 2 | ||
}; |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,16 @@ | ||
ARG grafana_version=latest | ||
ARG grafana_image=grafana-enterprise | ||
|
||
FROM grafana/${grafana_image}:${grafana_version} | ||
|
||
# Make it as simple as possible to access the grafana instance for development purposes | ||
# Do NOT enable these settings in a public facing / production grafana instance | ||
ENV GF_AUTH_ANONYMOUS_ORG_ROLE "Admin" | ||
ENV GF_AUTH_ANONYMOUS_ENABLED "true" | ||
ENV GF_AUTH_BASIC_ENABLED "false" | ||
# Set development mode so plugins can be loaded without the need to sign | ||
ENV GF_DEFAULT_APP_MODE "development" | ||
|
||
# Inject livereload script into grafana index.html | ||
USER root | ||
RUN sed -i 's/<\/body><\/html>/<script src=\"http:\/\/localhost:35729\/livereload.js\"><\/script><\/body><\/html>/g' /usr/share/grafana/public/views/index.html |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,164 @@ | ||
# Default build configuration by Grafana | ||
|
||
**This is an auto-generated directory and is not intended to be changed! ⚠️** | ||
|
||
The `.config/` directory holds basic configuration for the different tools | ||
that are used to develop, test and build the project. In order to make it updates easier we ask you to | ||
not edit files in this folder to extend configuration. | ||
|
||
## How to extend the basic configs? | ||
|
||
Bear in mind that you are doing it at your own risk, and that extending any of the basic configuration can lead | ||
to issues around working with the project. | ||
|
||
### Extending the ESLint config | ||
|
||
Edit the `.eslintrc` file in the project root in order to extend the ESLint configuration. | ||
|
||
**Example:** | ||
|
||
```json | ||
{ | ||
"extends": "./.config/.eslintrc", | ||
"rules": { | ||
"react/prop-types": "off" | ||
} | ||
} | ||
``` | ||
|
||
--- | ||
|
||
### Extending the Prettier config | ||
|
||
Edit the `.prettierrc.js` file in the project root in order to extend the Prettier configuration. | ||
|
||
**Example:** | ||
|
||
```javascript | ||
module.exports = { | ||
// Prettier configuration provided by Grafana scaffolding | ||
...require('./.config/.prettierrc.js'), | ||
|
||
semi: false, | ||
}; | ||
``` | ||
|
||
--- | ||
|
||
### Extending the Jest config | ||
|
||
There are two configuration in the project root that belong to Jest: `jest-setup.js` and `jest.config.js`. | ||
|
||
**`jest-setup.js`:** A file that is run before each test file in the suite is executed. We are using it to | ||
set up the Jest DOM for the testing library and to apply some polyfills. ([link to Jest docs](https://jestjs.io/docs/configuration#setupfilesafterenv-array)) | ||
|
||
**`jest.config.js`:** The main Jest configuration file that extends the Grafana recommended setup. ([link to Jest docs](https://jestjs.io/docs/configuration)) | ||
|
||
#### ESM errors with Jest | ||
|
||
A common issue found with the current jest config involves importing an npm package which only offers an ESM build. These packages cause jest to error with `SyntaxError: Cannot use import statement outside a module`. To work around this we provide a list of known packages to pass to the `[transformIgnorePatterns](https://jestjs.io/docs/configuration#transformignorepatterns-arraystring)` jest configuration property. If need be this can be extended in the following way: | ||
|
||
```javascript | ||
process.env.TZ = 'UTC'; | ||
const { grafanaESModules, nodeModulesToTransform } = require('./config/jest/utils'); | ||
|
||
module.exports = { | ||
// Jest configuration provided by Grafana | ||
...require('./.config/jest.config'), | ||
// Inform jest to only transform specific node_module packages. | ||
transformIgnorePatterns: [nodeModulesToTransform([...grafanaESModules, 'packageName'])], | ||
}; | ||
``` | ||
|
||
--- | ||
|
||
### Extending the TypeScript config | ||
|
||
Edit the `tsconfig.json` file in the project root in order to extend the TypeScript configuration. | ||
|
||
**Example:** | ||
|
||
```json | ||
{ | ||
"extends": "./.config/tsconfig.json", | ||
"compilerOptions": { | ||
"preserveConstEnums": true | ||
} | ||
} | ||
``` | ||
|
||
--- | ||
|
||
### Extending the Webpack config | ||
|
||
Follow these steps to extend the basic Webpack configuration that lives under `.config/`: | ||
|
||
#### 1. Create a new Webpack configuration file | ||
|
||
Create a new config file that is going to extend the basic one provided by Grafana. | ||
It can live in the project root, e.g. `webpack.config.ts`. | ||
|
||
#### 2. Merge the basic config provided by Grafana and your custom setup | ||
|
||
We are going to use [`webpack-merge`](https://github.com/survivejs/webpack-merge) for this. | ||
|
||
```typescript | ||
// webpack.config.ts | ||
import type { Configuration } from 'webpack'; | ||
import { merge } from 'webpack-merge'; | ||
import grafanaConfig from './.config/webpack/webpack.config'; | ||
|
||
const config = async (env): Promise<Configuration> => { | ||
const baseConfig = await grafanaConfig(env); | ||
|
||
return merge(baseConfig, { | ||
// Add custom config here... | ||
output: { | ||
asyncChunks: true, | ||
}, | ||
}); | ||
}; | ||
|
||
export default config; | ||
``` | ||
|
||
#### 3. Update the `package.json` to use the new Webpack config | ||
|
||
We need to update the `scripts` in the `package.json` to use the extended Webpack configuration. | ||
|
||
**Update for `build`:** | ||
|
||
```diff | ||
-"build": "webpack -c ./.config/webpack/webpack.config.ts --env production", | ||
+"build": "webpack -c ./webpack.config.ts --env production", | ||
``` | ||
|
||
**Update for `dev`:** | ||
|
||
```diff | ||
-"dev": "webpack -w -c ./.config/webpack/webpack.config.ts --env development", | ||
+"dev": "webpack -w -c ./webpack.config.ts --env development", | ||
``` | ||
|
||
### Configure grafana image to use when running docker | ||
|
||
By default `grafana-enterprise` will be used as the docker image for all docker related commands. If you want to override this behaviour simply alter the `docker-compose.yaml` by adding the following build arg `grafana_image`. | ||
|
||
**Example:** | ||
|
||
```yaml | ||
version: '3.7' | ||
|
||
services: | ||
grafana: | ||
container_name: 'myorg-basic-app' | ||
build: | ||
context: ./.config | ||
args: | ||
grafana_version: ${GRAFANA_VERSION:-9.1.2} | ||
grafana_image: ${GRAFANA_IMAGE:-grafana} | ||
``` | ||
|
||
In this example we are assigning the environment variable `GRAFANA_IMAGE` to the build arg `grafana_image` with a default value of `grafana`. This will give you the possibility to set the value while running the docker-compose commands which might be convinent in some scenarios. | ||
|
||
--- |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,25 @@ | ||
/* | ||
* ⚠️⚠️⚠️ THIS FILE WAS SCAFFOLDED BY `@grafana/create-plugin`. DO NOT EDIT THIS FILE DIRECTLY. ⚠️⚠️⚠️ | ||
* | ||
* In order to extend the configuration follow the steps in | ||
* https://grafana.github.io/plugin-tools/docs/advanced-configuration#extending-the-jest-config | ||
*/ | ||
|
||
import '@testing-library/jest-dom'; | ||
|
||
// https://jestjs.io/docs/manual-mocks#mocking-methods-which-are-not-implemented-in-jsdom | ||
Object.defineProperty(global, 'matchMedia', { | ||
writable: true, | ||
value: jest.fn().mockImplementation((query) => ({ | ||
matches: false, | ||
media: query, | ||
onchange: null, | ||
addListener: jest.fn(), // deprecated | ||
removeListener: jest.fn(), // deprecated | ||
addEventListener: jest.fn(), | ||
removeEventListener: jest.fn(), | ||
dispatchEvent: jest.fn(), | ||
})), | ||
}); | ||
|
||
HTMLCanvasElement.prototype.getContext = () => {}; |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,43 @@ | ||
/* | ||
* ⚠️⚠️⚠️ THIS FILE WAS SCAFFOLDED BY `@grafana/create-plugin`. DO NOT EDIT THIS FILE DIRECTLY. ⚠️⚠️⚠️ | ||
* | ||
* In order to extend the configuration follow the steps in | ||
* https://grafana.github.io/plugin-tools/docs/advanced-configuration#extending-the-jest-config | ||
*/ | ||
|
||
const path = require('path'); | ||
const { grafanaESModules, nodeModulesToTransform } = require('./jest/utils'); | ||
|
||
module.exports = { | ||
moduleNameMapper: { | ||
'\\.(css|scss|sass)$': 'identity-obj-proxy', | ||
'react-inlinesvg': path.resolve(__dirname, 'jest', 'mocks', 'react-inlinesvg.tsx'), | ||
}, | ||
modulePaths: ['<rootDir>/src'], | ||
setupFilesAfterEnv: ['<rootDir>/jest-setup.js'], | ||
testEnvironment: 'jest-environment-jsdom', | ||
testMatch: [ | ||
'<rootDir>/src/**/__tests__/**/*.{js,jsx,ts,tsx}', | ||
'<rootDir>/src/**/*.{spec,test,jest}.{js,jsx,ts,tsx}', | ||
'<rootDir>/src/**/*.{spec,test,jest}.{js,jsx,ts,tsx}', | ||
], | ||
transform: { | ||
'^.+\\.(t|j)sx?$': [ | ||
'@swc/jest', | ||
{ | ||
sourceMaps: 'inline', | ||
jsc: { | ||
parser: { | ||
syntax: 'typescript', | ||
tsx: true, | ||
decorators: false, | ||
dynamicImport: true, | ||
}, | ||
}, | ||
}, | ||
], | ||
}, | ||
// Jest will throw `Cannot use import statement outside module` if it tries to load an | ||
// ES module without it being transformed first. ./config/README.md#esm-errors-with-jest | ||
transformIgnorePatterns: [nodeModulesToTransform(grafanaESModules)], | ||
}; |
25 changes: 25 additions & 0 deletions
25
examples/app-with-service-account/.config/jest/mocks/react-inlinesvg.tsx
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,25 @@ | ||
// Due to the grafana/ui Icon component making fetch requests to | ||
// `/public/img/icon/<icon_name>.svg` we need to mock react-inlinesvg to prevent | ||
// the failed fetch requests from displaying errors in console. | ||
|
||
import React from 'react'; | ||
|
||
type Callback = (...args: any[]) => void; | ||
|
||
export interface StorageItem { | ||
content: string; | ||
queue: Callback[]; | ||
status: string; | ||
} | ||
|
||
export const cacheStore: { [key: string]: StorageItem } = Object.create(null); | ||
|
||
const SVG_FILE_NAME_REGEX = /(.+)\/(.+)\.svg$/; | ||
|
||
const InlineSVG = ({ src }: { src: string }) => { | ||
// testId will be the file name without extension (e.g. `public/img/icons/angle-double-down.svg` -> `angle-double-down`) | ||
const testId = src.replace(SVG_FILE_NAME_REGEX, '$2'); | ||
return <svg xmlns="http://www.w3.org/2000/svg" data-testid={testId} viewBox="0 0 24 24" />; | ||
}; | ||
|
||
export default InlineSVG; |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,29 @@ | ||
/* | ||
* ⚠️⚠️⚠️ THIS FILE WAS SCAFFOLDED BY `@grafana/create-plugin`. DO NOT EDIT THIS FILE DIRECTLY. ⚠️⚠️⚠️ | ||
* | ||
* In order to extend the configuration follow the steps in .config/README.md | ||
*/ | ||
|
||
/* | ||
* This utility function is useful in combination with jest `transformIgnorePatterns` config | ||
* to transform specific packages (e.g.ES modules) in a projects node_modules folder. | ||
*/ | ||
const nodeModulesToTransform = (moduleNames) => `node_modules\/(?!(${moduleNames.join('|')})\/)`; | ||
|
||
// Array of known nested grafana package dependencies that only bundle an ESM version | ||
const grafanaESModules = [ | ||
'.pnpm', // Support using pnpm symlinked packages | ||
'd3', | ||
'd3-color', | ||
'd3-force', | ||
'd3-interpolate', | ||
'd3-scale-chromatic', | ||
'ol', | ||
'react-colorful', | ||
'uuid', | ||
]; | ||
|
||
module.exports = { | ||
nodeModulesToTransform, | ||
grafanaESModules | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,26 @@ | ||
/* | ||
* ⚠️⚠️⚠️ THIS FILE WAS SCAFFOLDED BY `@grafana/create-plugin`. DO NOT EDIT THIS FILE DIRECTLY. ⚠️⚠️⚠️ | ||
* | ||
* In order to extend the configuration follow the steps in | ||
* https://grafana.github.io/plugin-tools/docs/advanced-configuration#extending-the-typescript-config | ||
*/ | ||
{ | ||
"compilerOptions": { | ||
"alwaysStrict": true, | ||
"declaration": false, | ||
"rootDir": "../src", | ||
"baseUrl": "../src", | ||
"typeRoots": ["../node_modules/@types"], | ||
"resolveJsonModule": true | ||
}, | ||
"ts-node": { | ||
"compilerOptions": { | ||
"module": "commonjs", | ||
"target": "es5", | ||
"esModuleInterop": true | ||
}, | ||
"transpileOnly": true | ||
}, | ||
"include": ["../src", "./types"], | ||
"extends": "@grafana/tsconfig" | ||
} |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are we actually meant to commit all these files? Should we git ignore them?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, we should include them. These are needed to execute the different
yarn
scripts.