Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Updating the asset group is not working #46831

Closed
1 of 2 tasks
atul-astronomer opened this issue Feb 17, 2025 · 2 comments · Fixed by #47090
Closed
1 of 2 tasks

Updating the asset group is not working #46831

atul-astronomer opened this issue Feb 17, 2025 · 2 comments · Fixed by #47090
Assignees
@uranusjr
Labels
affected_version:3.0.0alpha For all 3.0.0 alpha releases area:core area:datasets Issues related to the datasets feature kind:bug This is a clearly a bug priority:medium Bug that should be fixed before next release but would not block a release

Comments

@atul-astronomer
Copy link

atul-astronomer commented Feb 17, 2025 •
edited
Loading

Apache Airflow version

3.0.0a3

If "Other Airflow 2 version" selected, which one?

No response

What happened?

If user is trying the update the asset group which was already defined, then its not getting updated.

Asset definition while creation:
left = Asset(name='asset_name') - This goes by default in the asset group

Updating definition:
left = Asset(name='asset_name', group='Dataset') - This should update the asset group from asset to dataset.

Image Image Image

What you think should happen instead?

The asset group should be updated as per the asset definition

How to reproduce

  1. Create an asset where group is not specified. Asset(name='asset_name')
  2. Update the group parameter in the asset definition. Asset(name='asset_name', group='Dataset')
  3. Check the 'Get Asset 'API response and DB record in 'asset' table .

Operating System

Linux

Versions of Apache Airflow Providers

No response

Deployment

Other

Deployment details

No response

Anything else?

No response

Are you willing to submit PR?

  • Yes I am willing to submit a PR!

Code of Conduct
@atul-astronomer atul-astronomer added area:core kind:bug This is a clearly a bug needs-triage label for new issues that we didn't triage yet labels Feb 17, 2025
@atul-astronomer atul-astronomer changed the title Updating asset group is not working Updating the asset group is not working Feb 17, 2025
@dosubot dosubot bot added the area:datasets Issues related to the datasets feature label Feb 17, 2025
@vikramkoka vikramkoka added the affected_version:3.0.0alpha For all 3.0.0 alpha releases label Feb 17, 2025
@Lee-W Lee-W self-assigned this Feb 18, 2025
@vikramkoka vikramkoka removed the needs-triage label for new issues that we didn't triage yet label Feb 21, 2025
@vatsrahul1001 vatsrahul1001 added the priority:medium Bug that should be fixed before next release but would not block a release label Feb 25, 2025
@Lee-W Lee-W assigned uranusjr and unassigned Lee-W Feb 26, 2025
@uranusjr uranusjr mentioned this issue Feb 26, 2025
Merged
@prasad-madine
Copy link

Hello @atul-astronomer @Lee-W , Can I work on this issue?

@Lee-W
Copy link
Member

Lee-W commented Mar 5, 2025

It's already fixed

# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@uranusjr uranusjr

Labels
affected_version:3.0.0alpha For all 3.0.0 alpha releases area:core area:datasets Issues related to the datasets feature kind:bug This is a clearly a bug priority:medium Bug that should be fixed before next release but would not block a release
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Sync asset and alias property changes into db astronomer/airflow
6 participants
@uranusjr @Lee-W @vikramkoka @vatsrahul1001 @prasad-madine @atul-astronomer