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

Kusto: releasing 2021-08-27 client #3974

Merged
merged 28 commits into from
Oct 29, 2021
Merged

Conversation

zoharHenMicrosoft
Copy link
Member

@zoharHenMicrosoft zoharHenMicrosoft commented Oct 17, 2021

No description provided.

@zoharHenMicrosoft
Copy link
Member Author

@kairu-ms @jsntcy
Hi folks, I will appreciate your help with the failed tests above - everything passed locally so I'm not sure what is the issue here.
In addition - we are still getting the following alert: "The installed extension 'kusto' is experimental and not covered by customer support. Please use with discretion." - Can you please share which actions are required in order to be published it as GA?

@jsntcy jsntcy requested a review from kairu-ms October 18, 2021 02:43
azure-pipelines.yml Outdated Show resolved Hide resolved
@jsntcy
Copy link
Member

jsntcy commented Oct 18, 2021

@kairu-ms Kai Ru FTE @jsntcy Yu Chen FTE Hi folks, I will appreciate your help with the failed tests above - everything passed locally so I'm not sure what is the issue here. In addition - we are still getting the following alert: "The installed extension 'kusto' is experimental and not covered by customer support. Please use with discretion." - Can you please share which actions are required in order to be published it as GA?

  • Remove "azext.isExperimental": true, in azext_metadata.json so that the alert will disappear.
  • Regarding the CI failure, the error messages are obvious for you, you need fix them accordingly.

image

image

@zoharHenMicrosoft
Copy link
Member Author

@kairu-ms Kai Ru FTE @jsntcy Yu Chen FTE Hi folks, I will appreciate your help with the failed tests above - everything passed locally so I'm not sure what is the issue here. In addition - we are still getting the following alert: "The installed extension 'kusto' is experimental and not covered by customer support. Please use with discretion." - Can you please share which actions are required in order to be published it as GA?

  • Remove "azext.isExperimental": true, in azext_metadata.json so that the alert will disappear.
  • Regarding the CI failure, the error messages are obvious for you, you need fix them accordingly.

image

image

@yonzhan
Copy link
Collaborator

yonzhan commented Oct 18, 2021

Kusto

@yonzhan yonzhan added this to the Nov 2021 (2021-12-07) milestone Oct 18, 2021
@zoharHenMicrosoft
Copy link
Member Author

@kairu-ms Kai Ru FTE @jsntcy Yu Chen FTE Hi folks, I will appreciate your help with the failed tests above - everything passed locally so I'm not sure what is the issue here. In addition - we are still getting the following alert: "The installed extension 'kusto' is experimental and not covered by customer support. Please use with discretion." - Can you please share which actions are required in order to be published it as GA?

  • Remove "azext.isExperimental": true, in azext_metadata.json so that the alert will disappear.
  • Regarding the CI failure, the error messages are obvious for you, you need fix them accordingly.

image
image

@jsntcy sorry for being unclear, this is the error I can't solve. do you familiar with it?
image

@jsntcy
Copy link
Member

jsntcy commented Oct 21, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jsntcy
Copy link
Member

jsntcy commented Oct 22, 2021

@zoharHenMicrosoft, could you rebase your code on the latest main branch?

@zoharHenMicrosoft
Copy link
Member Author

@zoharHenMicrosoft, could you rebase your code on the latest main branch?

done

@zoharHenMicrosoft
Copy link
Member Author

@jsntcy @kairu-ms kindly reminder

@jsntcy
Copy link
Member

jsntcy commented Oct 26, 2021

锟斤拷<�!�D�O�C�T�Y�P�E� �h�t�m�l�>�

Remove this file.


Refers to: UpgradeLog.htm:1 in 202243f. [](commit_id = 202243f, deletion_comment = False)

@zoharHenMicrosoft
Copy link
Member Author

锟斤拷<�!�D�O�C�T�Y�P�E� �h�t�m�l�>�

Remove this file.

Refers to: UpgradeLog.htm:1 in 202243f. [](commit_id = 202243f, deletion_comment = False)

image

@jsntcy related?

@jsntcy
Copy link
Member

jsntcy commented Oct 26, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jsntcy
Copy link
Member

jsntcy commented Oct 26, 2021

锟斤拷<�!�D�O�C�T�Y�P�E� �h�t�m�l�>�

Remove this file.
Refers to: UpgradeLog.htm:1 in 202243f. [](commit_id = 202243f, deletion_comment = False)

image

@jsntcy related?

Not related, please try to rebase on the latest main branch again to see if the CI can pass.

@zoharHenMicrosoft
Copy link
Member Author

锟斤拷<�!�D�O�C�T�Y�P�E� �h�t�m�l�>�

Remove this file.
Refers to: UpgradeLog.htm:1 in 202243f. [](commit_id = 202243f, deletion_comment = False)

image
@jsntcy related?

Not related, please try to rebase on the latest main branch again to see if the CI can pass.

same result as before

@zoharHenMicrosoft
Copy link
Member Author

zoharHenMicrosoft commented Oct 26, 2021

锟斤拷<�!�D�O�C�T�Y�P�E� �h�t�m�l�>�

Remove this file.
Refers to: UpgradeLog.htm:1 in 202243f. [](commit_id = 202243f, deletion_comment = False)

image
@jsntcy related?

Not related, please try to rebase on the latest main branch again to see if the CI can pass.

same result as before

@jsntcy to rebase again?

@zoharHenMicrosoft
Copy link
Member Author

锟斤拷<�!�D�O�C�T�Y�P�E� �h�t�m�l�>�

Remove this file.
Refers to: UpgradeLog.htm:1 in 202243f. [](commit_id = 202243f, deletion_comment = False)

image
@jsntcy related?

Not related, please try to rebase on the latest main branch again to see if the CI can pass.

same result as before

@jsntcy to rebase again?

@jsntcy @kairu-ms Hi, do you have any estimation of when this bug will be solved? we need to merge this PR ASAP :)

@kairu-ms
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@zoharHenMicrosoft
Copy link
Member Author

/azp run

@jsntcy @kairu-ms all pass! thanks! any additional help from my side?

@kairu-ms kairu-ms merged commit 8361056 into Azure:main Oct 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants