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.

Sign up for GitHub

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

Jump to bottom

(test)[auth]Add call auth module case #41802

Closed
wants to merge 3 commits into from
Closed

(test)[auth]Add call auth module case #41802

wants to merge 3 commits into from

Conversation

zfr9527
Copy link
Contributor

@zfr9527 zfr9527 commented Oct 14, 2024

Proposed changes

(test)[auth]Add call auth module case

@doris-robot
Copy link

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR

Since 2024-03-18, the Document has been moved to doris-website.
See Doris Document.

@zfr9527
Copy link
Contributor Author

zfr9527 commented Oct 14, 2024

run buildall

@github-actions GitHub Actions
Copy link
Contributor

sh-checker report

To get the full details, please check in the job output.

shellcheck errors 

'shellcheck ' returned error 1 finding the following syntactical issues:

----------

In regression-test/data/auth_call/multi_load_cm.sh line 1:

^-- SC2148 (error): Tips depend on target shell and yours is unknown. Add a shebang or a 'shell' directive.


In regression-test/data/auth_call/stream_load_cm.sh line 1:

^-- SC2148 (error): Tips depend on target shell and yours is unknown. Add a shebang or a 'shell' directive.

For more information:
  https://www.shellcheck.net/wiki/SC2148 -- Tips depend on target shell and y...
----------

You can address the above issues in one of three ways:
1. Manually correct the issue in the offending shell script;
2. Disable specific issues by adding the comment:
  # shellcheck disable=NNNN
above the line that contains the issue, where NNNN is the error code;
3. Add '-e NNNN' to the SHELLCHECK_OPTS setting in your .yml action file.
shfmt errors 

'shfmt ' returned error 1 finding the following formatting issues:

----------
--- regression-test/data/auth_call/multi_load_cm.sh.orig
+++ regression-test/data/auth_call/multi_load_cm.sh
@@ -0,0 +1,1 @@
+
--- regression-test/data/auth_call/stream_load_cm.sh.orig
+++ regression-test/data/auth_call/stream_load_cm.sh
@@ -0,0 +1,1 @@
+
----------

You can reformat the above files to meet shfmt's requirements by typing:

  shfmt  -w filename

@github-actions GitHub Actions
Copy link
Contributor

sh-checker report

To get the full details, please check in the job output.

shellcheck errors 

'shellcheck ' returned error 1 finding the following syntactical issues:

----------

In regression-test/data/auth_call/multi_load_cm.sh line 1:

^-- SC2148 (error): Tips depend on target shell and yours is unknown. Add a shebang or a 'shell' directive.


In regression-test/data/auth_call/stream_load_cm.sh line 1:

^-- SC2148 (error): Tips depend on target shell and yours is unknown. Add a shebang or a 'shell' directive.

For more information:
  https://www.shellcheck.net/wiki/SC2148 -- Tips depend on target shell and y...
----------

You can address the above issues in one of three ways:
1. Manually correct the issue in the offending shell script;
2. Disable specific issues by adding the comment:
  # shellcheck disable=NNNN
above the line that contains the issue, where NNNN is the error code;
3. Add '-e NNNN' to the SHELLCHECK_OPTS setting in your .yml action file.
shfmt errors 

'shfmt ' returned error 1 finding the following formatting issues:

----------
--- regression-test/data/auth_call/multi_load_cm.sh.orig
+++ regression-test/data/auth_call/multi_load_cm.sh
@@ -0,0 +1,1 @@
+
--- regression-test/data/auth_call/stream_load_cm.sh.orig
+++ regression-test/data/auth_call/stream_load_cm.sh
@@ -0,0 +1,1 @@
+
----------

You can reformat the above files to meet shfmt's requirements by typing:

  shfmt  -w filename

@zfr9527 zfr9527 force-pushed the zfr_auth_add_case_1 branch 2 times, most recently from ad9bc68 to 8f55309 Compare October 14, 2024 09:22
@zfr9527
Copy link
Contributor Author

zfr9527 commented Oct 14, 2024

run buildall

@github-actions GitHub Actions
Copy link
Contributor

sh-checker report

To get the full details, please check in the job output.

shellcheck errors 

'shellcheck ' returned error 1 finding the following syntactical issues:

----------

In regression-test/data/auth_call/multi_load_cm.sh line 1:
# Licensed to the Apache Software Foundation (ASF) under one
^-- SC2148 (error): Tips depend on target shell and yours is unknown. Add a shebang or a 'shell' directive.


In regression-test/data/auth_call/stream_load_cm.sh line 1:
# Licensed to the Apache Software Foundation (ASF) under one
^-- SC2148 (error): Tips depend on target shell and yours is unknown. Add a shebang or a 'shell' directive.

For more information:
  https://www.shellcheck.net/wiki/SC2148 -- Tips depend on target shell and y...
----------

You can address the above issues in one of three ways:
1. Manually correct the issue in the offending shell script;
2. Disable specific issues by adding the comment:
  # shellcheck disable=NNNN
above the line that contains the issue, where NNNN is the error code;
3. Add '-e NNNN' to the SHELLCHECK_OPTS setting in your .yml action file.
shfmt errors 
'shfmt ' found no issues.

@github-actions GitHub Actions
Copy link
Contributor

sh-checker report

To get the full details, please check in the job output.

shellcheck errors 

'shellcheck ' returned error 1 finding the following syntactical issues:

----------

In regression-test/data/auth_call/multi_load_cm.sh line 1:
# Licensed to the Apache Software Foundation (ASF) under one
^-- SC2148 (error): Tips depend on target shell and yours is unknown. Add a shebang or a 'shell' directive.


In regression-test/data/auth_call/stream_load_cm.sh line 1:
# Licensed to the Apache Software Foundation (ASF) under one
^-- SC2148 (error): Tips depend on target shell and yours is unknown. Add a shebang or a 'shell' directive.

For more information:
  https://www.shellcheck.net/wiki/SC2148 -- Tips depend on target shell and y...
----------

You can address the above issues in one of three ways:
1. Manually correct the issue in the offending shell script;
2. Disable specific issues by adding the comment:
  # shellcheck disable=NNNN
above the line that contains the issue, where NNNN is the error code;
3. Add '-e NNNN' to the SHELLCHECK_OPTS setting in your .yml action file.
shfmt errors 
'shfmt ' found no issues.

@zfr9527 zfr9527 force-pushed the zfr_auth_add_case_1 branch 2 times, most recently from 5df8e1c to 48566ee Compare October 14, 2024 09:42
@zfr9527 zfr9527 force-pushed the zfr_auth_add_case_1 branch 2 times, most recently from fba72a3 to ba8f6b5 Compare October 15, 2024 06:29
@zddr zddr mentioned this pull request Oct 18, 2024
Merged
morningman pushed a commit that referenced this pull request Oct 18, 2024
@zddr
[fix](auth)fix show load priv bug (#41723)
0c2901b 
fix when user have `load_priv` on table,but can not show load about this
table

ps: We will add cases to other PRs as a
whole:#41802
zddr added a commit to zddr/incubator-doris that referenced this pull request Oct 18, 2024
@zddr
[fix](auth)fix show load priv bug (apache#41723)
91752fd 
fix when user have `load_priv` on table,but can not show load about this
table

ps: We will add cases to other PRs as a
whole:apache#41802
zddr added a commit to zddr/incubator-doris that referenced this pull request Oct 18, 2024
@zddr
[fix](auth)fix show load priv bug (apache#41723)
492cc1c 
fix when user have `load_priv` on table,but can not show load about this
table

ps: We will add cases to other PRs as a
whole:apache#41802
zddr added a commit to zddr/incubator-doris that referenced this pull request Oct 18, 2024
@zddr
[fix](auth)fix show load priv bug (apache#41723)
fb5c8f0 
fix when user have `load_priv` on table,but can not show load about this
table

ps: We will add cases to other PRs as a
whole:apache#41802
morningman pushed a commit to morningman/doris that referenced this pull request Oct 21, 2024
@zddr @morningman
[fix](auth)fix show load priv bug (apache#41723)
fda7ba0 
fix when user have `load_priv` on table,but can not show load about this
table

ps: We will add cases to other PRs as a
whole:apache#41802
@zfr9527 zfr9527 closed this Oct 23, 2024
@zfr9527
Copy link
Contributor Author

zfr9527 commented Oct 23, 2024

There is a conflict. After resolving it, a new pull request has been submitted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@zfr9527 @doris-robot