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

fix: app failing to mount with topologyconstrainedpools and volumenameprefix #4003

Closed
wants to merge 2 commits into from
Closed

fix: app failing to mount with topologyconstrainedpools and volumenameprefix #4003

wants to merge 2 commits into from

Conversation

riya-singhal31
Copy link
Contributor

Describe what this PR does

Provide some context for the reviewer

Is there anything that requires special attention

Do you have any questions?

Is the change backward compatible?

Are there concerns around backward compatibility?

Provide any external context for the change, if any.

For example:

  • Kubernetes links that explain why the change is required
  • CSI spec related changes/catch-up that necessitates this patch
  • golang related practices that necessitates this change

Related issues

Mention any github issues relevant to this PR. Adding below line
will help to auto close the issue once the PR is merged.

Fixes: #issue_number

Future concerns

List items that are not part of the PR and do not impact it's
functionality, but are work items that can be taken up subsequently.


Show available bot commands

These commands are normally not required, but in case of issues, leave any of
the following bot commands in an otherwise empty comment in this PR:

  • /retest ci/centos/<job-name>: retest the <job-name> after unrelated
    failure (please report the failure too!)

@mergify mergify bot added the bug Something isn't working label Jul 19, 2023
@riya-singhal31 riya-singhal31 changed the title fix: app failing to mount with volumenameprefix fix: app failing to mount with topologyconstrainedpools and volumenameprefix Jul 19, 2023
@riya-singhal31
Copy link
Contributor Author

/test ci/centos/mini-e2e-helm/k8s-1.27

1 similar comment
@riya-singhal31
Copy link
Contributor Author

/test ci/centos/mini-e2e-helm/k8s-1.27

@riya-singhal31
Copy link
Contributor Author

/test ci/centos/mini-e2e-helm/k8s-1.27

@riya-singhal31
Copy link
Contributor Author

/test ci/centos/mini-e2e-helm/k8s-1.27

@riya-singhal31
Copy link
Contributor Author

/test ci/centos/mini-e2e-helm/k8s-1.28

@github-actions
Copy link

github-actions bot commented Oct 4, 2023

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in two weeks if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale label Oct 4, 2023
@Madhu-1
Copy link
Collaborator

Madhu-1 commented Oct 5, 2023

@riya-singhal31 any plan to update this PR?

@riya-singhal31
Copy link
Contributor Author

@riya-singhal31 any plan to update this PR?

Yes Madhu, I am just busy with few testing and updating of the Fencing PR, once that is done, I'll get it done.

@Madhu-1 Madhu-1 removed the stale label Oct 5, 2023
Copy link

github-actions bot commented Nov 4, 2023

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in two weeks if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale label Nov 4, 2023
@riya-singhal31 riya-singhal31 closed this by deleting the head repository Nov 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working component/rbd Issues related to RBD Not Ready For Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants