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

sample_glue_loader returns 'Something went wrong...' on the front-end #881

Closed
eponkratova opened this issue Jan 17, 2021 · 18 comments
Closed
Labels
type:bug An unexpected problem or unintended behavior

Comments

@eponkratova
Copy link

Expected Behavior

Glue tables metadata is displayed on the front-end

Current Behavior

On the front-end, I see 'Something went wrong...'

Possible Solution

Steps to Reproduce

I am running Amundsen on my local machine:

  1. followed the installation guide at https://github.com/amundsen-io/amundsen/blob/master/docs/installation.md, including the required python libraries.
  2. modified sample_glue_loader and added aws session token.
  3. when I opened the front-end, the tables from Glue were listed but when I click any, the error displayed 'Something went wrong...' . The example of the URL of one of the tables (iriscsv) located in in the iris database is:
    http://localhost:5000/table_detail/AWSSessionToken=%22/glue/iris/iriscsv.

Screenshots (if appropriate)

Context

I am doing a POC.

Your Environment

@feng-tao
Copy link
Member

do you verify the table is ingested into neo4j correctly with neo4j UI?

@feng-tao feng-tao added the type:question An issue or PR that needs more information or a user question label Jan 22, 2021
@eponkratova
Copy link
Author

eponkratova commented Jan 23, 2021

I can see the tables on neo4j, when I switch to the table view, I see the similar content for all tables retrieved:
{
"name": "iriscsv",
"publisher_last_updated_epoch_ms": 1610903614689,
"published_tag": "1610874811",
"key": "glue://sessiontoken=".iris/iriscsv",
"is_view": false
}

@OElesin
Copy link

OElesin commented Jan 30, 2021

I'm experiencing the same thing as well. Any help here

@eponkratova
Copy link
Author

OElesin, check the Athena one if you need to get crawl the s3 data. If you use Python, not Airflow, I slightly modified the original code at https://github.com/eponkratova/articles/blob/master/data_discovery/code/athena_data_loader.py

@OElesin
Copy link

OElesin commented Jan 31, 2021

@eponkratova, I tried this and got the same error. This is quite surprising .

I executed the API request from the browser console and got the error below:

{
"msg": "Encountered error: Metadata request failed",
"status_code": 500,
"tableData": {}
}

@OElesin
Copy link

OElesin commented Jan 31, 2021

From the error logs

self.client = get_proxy_client()
--
File "/usr/local/lib/python3.7/site-packages/amundsen_metadata-3.0.1-py3.7.egg/metadata_service/proxy/__init__.py", line 47, in get_proxy_client
client_kwargs=client_kwargs)
TypeError: __init__() got an unexpected keyword argument 'client_kwargs'

@OElesin
Copy link

OElesin commented Jan 31, 2021

@feng-tao and @eponkratova, the issue is likely due to recent changes in the metadata library. I reverted the image being used to amundsen-metadata:3.0.0 and everything seems to be fixed.

Looks like this is from: https://github.com/amundsen-io/amundsenmetadatalibrary/blob/master/metadata_service/proxy/__init__.py#L47

@eponkratova
Copy link
Author

@OElesin , so, did you make the Glue connector working as well?

@OElesin
Copy link

OElesin commented Feb 1, 2021

@eponkratova, both are working.

@eponkratova
Copy link
Author

Aha, thank you, @OElesin

@feng-tao
Copy link
Member

feng-tao commented Feb 1, 2021

cc @mgorsk1

@NiravLangaliya
Copy link

I have installed new code and I am also facing the same issue and it complains about

TypeError: __init__() got an unexpected keyword argument 'client_kwargs'

@feng-tao
Copy link
Member

feng-tao commented Feb 4, 2021

@mgorsk1 could you take a look?

@feng-tao feng-tao added type:bug An unexpected problem or unintended behavior Project: Metadata and removed type:question An issue or PR that needs more information or a user question labels Feb 4, 2021
@mgorsk1
Copy link
Contributor

mgorsk1 commented Feb 4, 2021

sure, looking into it

@mgorsk1
Copy link
Contributor

mgorsk1 commented Feb 4, 2021

apologies, that's my blunder. I've submitted fix amundsen-io/amundsenmetadatalibrary#258 any chance you can verify if that helped @OElesin @eponkratova

@feng-tao
Copy link
Member

feng-tao commented Feb 4, 2021

no worry, @mgorsk1 's change (amundsen-io/amundsenmetadatalibrary#258) is just merged. @NiravLangaliya could you pull from master and see if the issue is gone?

@feng-tao
Copy link
Member

feng-tao commented Feb 5, 2021

Feel free to reopen if the issue persists after pulling from master.

@feng-tao feng-tao closed this as completed Feb 5, 2021
dorianj pushed a commit to dorianj/amundsen that referenced this issue Apr 25, 2021
…undsen-io#881)

* added PR creation to release action

Signed-off-by: Allison Suarez Miranda <[email protected]>

* set new branch rather than master

Signed-off-by: Allison Suarez Miranda <[email protected]>

* changed cron to 12:15pm

Signed-off-by: Allison Suarez Miranda <[email protected]>

* chnaged release aciton branch and added manual run trigger

Signed-off-by: Allison Suarez Miranda <[email protected]>

* eventually I'll get this right...

Signed-off-by: Allison Suarez Miranda <[email protected]>

* another one

Signed-off-by: Allison Suarez Miranda <[email protected]>

* one more branch change

Signed-off-by: Allison Suarez Miranda <[email protected]>

* trying to get the branch there so it can be checked out

Signed-off-by: Allison Suarez Miranda <[email protected]>

* added pypy credentials

Signed-off-by: Allison Suarez Miranda <[email protected]>

* chaged to uses/with

Signed-off-by: Allison Suarez Miranda <[email protected]>

* bumped version on setup.py file

Signed-off-by: Allison Suarez Miranda <[email protected]>

* added pypi user and pass

Signed-off-by: Allison Suarez Miranda <[email protected]>
feng-tao pushed a commit that referenced this issue May 7, 2021
* added PR creation to release action

Signed-off-by: Allison Suarez Miranda <[email protected]>

* set new branch rather than master

Signed-off-by: Allison Suarez Miranda <[email protected]>

* changed cron to 12:15pm

Signed-off-by: Allison Suarez Miranda <[email protected]>

* chnaged release aciton branch and added manual run trigger

Signed-off-by: Allison Suarez Miranda <[email protected]>

* eventually I'll get this right...

Signed-off-by: Allison Suarez Miranda <[email protected]>

* another one

Signed-off-by: Allison Suarez Miranda <[email protected]>

* one more branch change

Signed-off-by: Allison Suarez Miranda <[email protected]>

* trying to get the branch there so it can be checked out

Signed-off-by: Allison Suarez Miranda <[email protected]>

* added pypy credentials

Signed-off-by: Allison Suarez Miranda <[email protected]>

* chaged to uses/with

Signed-off-by: Allison Suarez Miranda <[email protected]>

* bumped version on setup.py file

Signed-off-by: Allison Suarez Miranda <[email protected]>

* added pypi user and pass

Signed-off-by: Allison Suarez Miranda <[email protected]>
@cpollard0
Copy link

@feng-tao - I'm getting the same issue now with loading Glue data. I'm on the latest branch from master, my data is in neo4j. When I click on a table in the WebUI I get 404 not found. Any guidance would be appreciated!

zacr pushed a commit to SaltIO/amundsen that referenced this issue May 13, 2022
…undsen-io#881)

* added PR creation to release action

Signed-off-by: Allison Suarez Miranda <[email protected]>

* set new branch rather than master

Signed-off-by: Allison Suarez Miranda <[email protected]>

* changed cron to 12:15pm

Signed-off-by: Allison Suarez Miranda <[email protected]>

* chnaged release aciton branch and added manual run trigger

Signed-off-by: Allison Suarez Miranda <[email protected]>

* eventually I'll get this right...

Signed-off-by: Allison Suarez Miranda <[email protected]>

* another one

Signed-off-by: Allison Suarez Miranda <[email protected]>

* one more branch change

Signed-off-by: Allison Suarez Miranda <[email protected]>

* trying to get the branch there so it can be checked out

Signed-off-by: Allison Suarez Miranda <[email protected]>

* added pypy credentials

Signed-off-by: Allison Suarez Miranda <[email protected]>

* chaged to uses/with

Signed-off-by: Allison Suarez Miranda <[email protected]>

* bumped version on setup.py file

Signed-off-by: Allison Suarez Miranda <[email protected]>

* added pypi user and pass

Signed-off-by: Allison Suarez Miranda <[email protected]>
hansadriaans pushed a commit to DataChefHQ/amundsen that referenced this issue Jun 30, 2022
…undsen-io#881)

* added PR creation to release action

Signed-off-by: Allison Suarez Miranda <[email protected]>

* set new branch rather than master

Signed-off-by: Allison Suarez Miranda <[email protected]>

* changed cron to 12:15pm

Signed-off-by: Allison Suarez Miranda <[email protected]>

* chnaged release aciton branch and added manual run trigger

Signed-off-by: Allison Suarez Miranda <[email protected]>

* eventually I'll get this right...

Signed-off-by: Allison Suarez Miranda <[email protected]>

* another one

Signed-off-by: Allison Suarez Miranda <[email protected]>

* one more branch change

Signed-off-by: Allison Suarez Miranda <[email protected]>

* trying to get the branch there so it can be checked out

Signed-off-by: Allison Suarez Miranda <[email protected]>

* added pypy credentials

Signed-off-by: Allison Suarez Miranda <[email protected]>

* chaged to uses/with

Signed-off-by: Allison Suarez Miranda <[email protected]>

* bumped version on setup.py file

Signed-off-by: Allison Suarez Miranda <[email protected]>

* added pypi user and pass

Signed-off-by: Allison Suarez Miranda <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:bug An unexpected problem or unintended behavior
Projects
None yet
Development

No branches or pull requests

6 participants