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

Update logging messages for JWT tokens #12169

Merged
merged 1 commit into from
Jun 27, 2023

Conversation

pmbrull
Copy link
Collaborator

@pmbrull pmbrull commented Jun 27, 2023

Describe your changes:

To make it easier to identify issues

Type of change:

  • Bug fix
  • Improvement
  • New feature
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation

Checklist:

  • I have read the CONTRIBUTING document.
  • My PR title is Fixes <issue-number>: <short explanation>
  • I have commented on my code, particularly in hard-to-understand areas.
  • For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

@sonarcloud
Copy link

sonarcloud bot commented Jun 27, 2023

[open-metadata-airflow-apis] Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@sonarcloud
Copy link

sonarcloud bot commented Jun 27, 2023

[open-metadata-ingestion] Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@cypress
Copy link

cypress bot commented Jun 27, 2023

4 failed tests on run #24925 ↗︎

4 245 59 0 Flakiness 0

Details:

Update logging messages for JWT tokens
Project: openmetadata Commit: 5a99e67cfe
Status: Failed Duration: 29:13 💡
Started: Jun 27, 2023 7:25 AM Ended: Jun 27, 2023 7:55 AM
Failed  cypress/e2e/Features/RestoreEntity.spec.js • 4 failed tests • Tests with database postgresql

View Output Video

Test Artifacts
Restore entity functionality should work properly > Soft Delete entity table Output Screenshots Video
Restore entity functionality should work properly > Check Soft Deleted entity table Output Screenshots Video
Restore entity functionality should work properly > Check Soft Deleted table in it's Schema Output Screenshots Video
Restore entity functionality should work properly > Restore Soft Deleted table Output Screenshots Video

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@pmbrull pmbrull merged commit c22c587 into open-metadata:main Jun 27, 2023
15 of 21 checks passed
@pmbrull pmbrull deleted the workflow-log branch June 27, 2023 10:32
mohityadav766 added a commit that referenced this pull request Jun 28, 2023
…disabled (#12144)

* Fix #10198: Filter tags and classifications if the classification is disabled

* Fix #10198: Fix tests

* Fix #10198: Fix tests

* Fix #10198: Fix Postgres tests

* Fix #10198: Fix Postgres tests

* Fix #12083 - Backoff retry when polling for pipeline service client status (#12105)

* Fix #12083

* Fix #12083

---------

Co-authored-by: Sriharsha Chintalapani <harshach@users.noreply.github.com>

* Databricks Support Table Constraints (#12138)

* Databricks Support Table Constraints

* pylint fix

---------

Co-authored-by: Pere Miquel Brull <peremiquelbrull@gmail.com>

* Fix #11659: Add support for filter patterns in dbt workflow (#12063)

* Move Top Level Imports (#12145)

* interfaces for db migrations (#12057)

* interfaces for db migrations

* Add Data Migration for entities to 1.1

* Migration for FQN hash

* Added Tag Usage And Fixed some transaction issue in testing

* Added SERVER_CHANGE_LOG validations

* Update Path for files

* Added Postgres Migration

* Test Suite Migration [WIP]

* remove ingestion pipeline

* moved Migration to Table Initializer

* init Data source config in TableInitializer

* make checksum ignore configurable

---------

Co-authored-by: mohitdeuex <mohit.y@deuexsolutions.com>

* Fix Migrations (#12146)

* Mask queries, Table sample data and Topic sample data (#12139)

* Filter out PII queries

* Mask topic PII

* Mask topic PII

* Update sample data test

* Format

* Moved logic to repository or PIIMasker

* Fix test

* chore(ui): update sample data api endpoint for topic

---------

Co-authored-by: Sachin Chaurasiya <sachinchaurasiyachotey87@gmail.com>

* Dockerfile Update (#11066)

* Updating Dockerfile with multistage

* Updating reviewed changes

* Docker Development changes

* Docker workflow changes

* Arguments update

* Script path update

* Arguments update

* Resolving the reviewed suggestions

* Docker script update (#12115)

* Create manual.yml

* Fixing Typo

* Removing manual test github action file

* Testing IP runner

* Updating docker script for sandbox db task

* removing the ip test action

* Updating docker script for sandbox db task postgres

* Volume cleaning argument addition

* Volume cleaning argument addition

* Fix Failing Maven CI Test (#12151)

* Fix SQLLineage Test (#12152)

* Docker server workflow (#12157)

* Updating Dockerfile with multistage

* Updating reviewed changes

* Docker Development changes

* Docker workflow changes

* Arguments update

* Script path update

* Arguments update

* Resolving the reviewed suggestions

* updating workflow syntax

* remvoed ibm db arch (#12158)

* fix: resolve general feedbacks (#12153)

* fix: feedbacks

* fix: add tier as static item

* fix: feedbacks

* fix: add localization

* fix: feedbacks

* fix: add localization

* fix: breadcrumb wrapping issue

* fix: alignment issue for glossary

* fix: border issue in drawer

* ui: worked on DQ feedback part 3 (#12149)

* ui: worked on DQ feedback part 3

* updated name of test suite ingestion name
- Added description field in test case expand view
- Added functionality to update displayname of test case

* added testCase in re-indexing

* integrated permission in DQ

* fix: check on undefined bool value of clause (#12162)

* chore(ui): task feedback (#12154)

* chore(ui): task feedback

* fix copy notification info

* fix code font

* fix glue pipeline

* fix unit tests

* fix cy failures

* Update CODEOWNERS for UI code

* Address backend DQ feedback (#12150)

* fix: add jp and zh index

* fix: updated testCase tooltips

* fix: changed list testSuite permission to VEW_TEST + set owner of table for executable testSuite

* Run System Migrations on CREATE

* Update logging messages for JWT tokens (#12169)

* Update Test Case Activity Feed Messaging

* Auto tagging Windows troubleshooting (#12173)

* feat(ui): supported pagination and search in tags and term select for both table and entity level (#12155)

* supported pagination and search in tags and term select for both table and entity level

* supported pagination of tags in glossary overiew section for tags

* fix cypress and address comments

* fix cypress issue

* fixed(ui): updated tour flow and feedback (#12171)

* fixed(ui): updated tour flow and feedback

* translation-sync

* addressing comments

* feat: added view definition tab in table details page (#12172)

* Update profiler dq doc (#12182)

* doc: update documentation for profiler and dq

* doc: renamed image path from 1.0.0 to 1.1.0

* fix connection kwargs error in profiler (#12181)

* Fix Profiler Workflow infinite loop (#12185)

* Fix Profiler Workflow infinite loop

* fix: scrolling behavior feedback on mydata page (#12176)

* fix(ui): task layout update (#12175)

* fix(ui): task layout update

* fix title for edit task

* remove local for edit-task

* feat(ui): supported pagination of glossary terms in glossary overview section (#12177)

* supported pagination of glossary terms in glossay overview section

* remove unwanted try catch

* change as per comments

* Update S3 storage permissions (#12064)

* Add disabled flag to search mappings

* Revert unwanted changes

---------

Co-authored-by: Pere Miquel Brull <peremiquelbrull@gmail.com>
Co-authored-by: Mayur Singal <39544459+ulixius9@users.noreply.github.com>
Co-authored-by: Ayush Shah <ayush@getcollate.io>
Co-authored-by: mohitdeuex <mohit.y@deuexsolutions.com>
Co-authored-by: Sachin Chaurasiya <sachinchaurasiyachotey87@gmail.com>
Co-authored-by: Anuj359 <anuj.j@deuexsolutions.com>
Co-authored-by: Mohit Yadav <105265192+mohityadav766@users.noreply.github.com>
Co-authored-by: Onkar Ravgan <onkar.10r@gmail.com>
Co-authored-by: karanh37 <33024356+karanh37@users.noreply.github.com>
Co-authored-by: Shailesh Parmar <shailesh.parmar.webdev@gmail.com>
Co-authored-by: Teddy <teddy.crepineau@gmail.com>
Co-authored-by: Chirag Madlani <12962843+chirag-madlani@users.noreply.github.com>
Co-authored-by: Ashish Gupta <ashish@getcollate.io>
harshach added a commit that referenced this pull request Jun 30, 2023
…disabled (#12144)

* Fix #10198: Filter tags and classifications if the classification is disabled

* Fix #10198: Fix tests

* Fix #10198: Fix tests

* Fix #10198: Fix Postgres tests

* Fix #10198: Fix Postgres tests

* Fix #12083 - Backoff retry when polling for pipeline service client status (#12105)

* Fix #12083

* Fix #12083

---------

Co-authored-by: Sriharsha Chintalapani <harshach@users.noreply.github.com>

* Databricks Support Table Constraints (#12138)

* Databricks Support Table Constraints

* pylint fix

---------

Co-authored-by: Pere Miquel Brull <peremiquelbrull@gmail.com>

* Fix #11659: Add support for filter patterns in dbt workflow (#12063)

* Move Top Level Imports (#12145)

* interfaces for db migrations (#12057)

* interfaces for db migrations

* Add Data Migration for entities to 1.1

* Migration for FQN hash

* Added Tag Usage And Fixed some transaction issue in testing

* Added SERVER_CHANGE_LOG validations

* Update Path for files

* Added Postgres Migration

* Test Suite Migration [WIP]

* remove ingestion pipeline

* moved Migration to Table Initializer

* init Data source config in TableInitializer

* make checksum ignore configurable

---------

Co-authored-by: mohitdeuex <mohit.y@deuexsolutions.com>

* Fix Migrations (#12146)

* Mask queries, Table sample data and Topic sample data (#12139)

* Filter out PII queries

* Mask topic PII

* Mask topic PII

* Update sample data test

* Format

* Moved logic to repository or PIIMasker

* Fix test

* chore(ui): update sample data api endpoint for topic

---------

Co-authored-by: Sachin Chaurasiya <sachinchaurasiyachotey87@gmail.com>

* Dockerfile Update (#11066)

* Updating Dockerfile with multistage

* Updating reviewed changes

* Docker Development changes

* Docker workflow changes

* Arguments update

* Script path update

* Arguments update

* Resolving the reviewed suggestions

* Docker script update (#12115)

* Create manual.yml

* Fixing Typo

* Removing manual test github action file

* Testing IP runner

* Updating docker script for sandbox db task

* removing the ip test action

* Updating docker script for sandbox db task postgres

* Volume cleaning argument addition

* Volume cleaning argument addition

* Fix Failing Maven CI Test (#12151)

* Fix SQLLineage Test (#12152)

* Docker server workflow (#12157)

* Updating Dockerfile with multistage

* Updating reviewed changes

* Docker Development changes

* Docker workflow changes

* Arguments update

* Script path update

* Arguments update

* Resolving the reviewed suggestions

* updating workflow syntax

* remvoed ibm db arch (#12158)

* fix: resolve general feedbacks (#12153)

* fix: feedbacks

* fix: add tier as static item

* fix: feedbacks

* fix: add localization

* fix: feedbacks

* fix: add localization

* fix: breadcrumb wrapping issue

* fix: alignment issue for glossary

* fix: border issue in drawer

* ui: worked on DQ feedback part 3 (#12149)

* ui: worked on DQ feedback part 3

* updated name of test suite ingestion name
- Added description field in test case expand view
- Added functionality to update displayname of test case

* added testCase in re-indexing

* integrated permission in DQ

* fix: check on undefined bool value of clause (#12162)

* chore(ui): task feedback (#12154)

* chore(ui): task feedback

* fix copy notification info

* fix code font

* fix glue pipeline

* fix unit tests

* fix cy failures

* Update CODEOWNERS for UI code

* Address backend DQ feedback (#12150)

* fix: add jp and zh index

* fix: updated testCase tooltips

* fix: changed list testSuite permission to VEW_TEST + set owner of table for executable testSuite

* Run System Migrations on CREATE

* Update logging messages for JWT tokens (#12169)

* Update Test Case Activity Feed Messaging

* Auto tagging Windows troubleshooting (#12173)

* feat(ui): supported pagination and search in tags and term select for both table and entity level (#12155)

* supported pagination and search in tags and term select for both table and entity level

* supported pagination of tags in glossary overiew section for tags

* fix cypress and address comments

* fix cypress issue

* fixed(ui): updated tour flow and feedback (#12171)

* fixed(ui): updated tour flow and feedback

* translation-sync

* addressing comments

* feat: added view definition tab in table details page (#12172)

* Update profiler dq doc (#12182)

* doc: update documentation for profiler and dq

* doc: renamed image path from 1.0.0 to 1.1.0

* fix connection kwargs error in profiler (#12181)

* Fix Profiler Workflow infinite loop (#12185)

* Fix Profiler Workflow infinite loop

* fix: scrolling behavior feedback on mydata page (#12176)

* fix(ui): task layout update (#12175)

* fix(ui): task layout update

* fix title for edit task

* remove local for edit-task

* feat(ui): supported pagination of glossary terms in glossary overview section (#12177)

* supported pagination of glossary terms in glossay overview section

* remove unwanted try catch

* change as per comments

* Update S3 storage permissions (#12064)

* Add disabled flag to search mappings

* Revert unwanted changes

---------

Co-authored-by: Pere Miquel Brull <peremiquelbrull@gmail.com>
Co-authored-by: Mayur Singal <39544459+ulixius9@users.noreply.github.com>
Co-authored-by: Ayush Shah <ayush@getcollate.io>
Co-authored-by: mohitdeuex <mohit.y@deuexsolutions.com>
Co-authored-by: Sachin Chaurasiya <sachinchaurasiyachotey87@gmail.com>
Co-authored-by: Anuj359 <anuj.j@deuexsolutions.com>
Co-authored-by: Mohit Yadav <105265192+mohityadav766@users.noreply.github.com>
Co-authored-by: Onkar Ravgan <onkar.10r@gmail.com>
Co-authored-by: karanh37 <33024356+karanh37@users.noreply.github.com>
Co-authored-by: Shailesh Parmar <shailesh.parmar.webdev@gmail.com>
Co-authored-by: Teddy <teddy.crepineau@gmail.com>
Co-authored-by: Chirag Madlani <12962843+chirag-madlani@users.noreply.github.com>
Co-authored-by: Ashish Gupta <ashish@getcollate.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ingestion safe to test Add this label to run secure Github workflows on PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants